feat: added output management integration#161
Conversation
|
|
|
Hi. Could you give a more detailed description of this new feature for future reference? Also, is necessary that the title follows commit convention, for exemple: |
SDK Module Review
Findings (4)
4 finding(s): 1 posted as inline comment(s) on the affected lines, 3 not tied to a code line (listed above). Generated by sdk-review-skill · v1 |
| f"Failed to create destination client for instance '{inst}'. " | ||
| f"Ensure the Destination Service is properly bound and configured." | ||
| ) from e | ||
| if self.access_strategy: |
There was a problem hiding this comment.
Why not use enum directly?
| strategy = AccessStrategy.PROVIDER_ONLY | ||
| else: | ||
| strategy = AccessStrategy.SUBSCRIBER_ONLY | ||
| destination = client.get_subaccount_destination( |
There was a problem hiding this comment.
Isn't this tenant specific destination?
Description
Output Management is a BTP shared service which has features like sending documents to external system in various channels like EDI/EMAIL/Print. As part of this, we are mainly concentrating on the email use case where one can send email via our framework along with attachments if necessary.
We are exposing capabilities via util methods of sending email as BTP destination based and via integrated MCP hub. When using the MCP Hub way, we are giving the capability to construct the required payload and invoke the util functions. When using the BTP destination way, we are giving the options to input the destination details and the payloads.
Related Issue
Closes #<issue_number>
(Link to the GitHub issue this PR addresses)
Type of Change
Please check the relevant option:
How to Test
Describe how reviewers can test your changes:
Checklist
Before submitting your PR, please review and check the following:
Breaking Changes
If this PR introduces breaking changes, please describe:
Additional Notes
Add any additional context, screenshots, or information that would help reviewers.