-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cds^8.2] Event Broker: S/4HANA Cloud to CAP #1198
Conversation
guides/messaging/event-broker.md
Outdated
// TODO: the guide is not yet published!!! | ||
|
||
Follow guide _SAP Event Broker Service Guide_ → _Integration Scenarios_ → [CAP Application as a Subscriber](https://help.sap.com/docs/event-broker/event-broker-draft-service/integration-example-using-cap-application?state=DRAFT) to prepare your SAP BTP account for event consumption. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// TODO: the guide is not yet published!!! | |
Follow guide _SAP Event Broker Service Guide_ → _Integration Scenarios_ → [CAP Application as a Subscriber](https://help.sap.com/docs/event-broker/event-broker-draft-service/integration-example-using-cap-application?state=DRAFT) to prepare your SAP BTP account for event consumption. | |
Follow guide _SAP Event Broker Service Guide_ → _Integration Scenarios_ → [CAP Application as a Subscriber](https://help.sap.com/docs/event-broker/event-broker-draft-service/integration-example-using-cap-application) to prepare your SAP BTP account for event consumption. |
@renejeglinsky @smahati we need to replace with a link to a published doc before releasing
guides/messaging/event-broker.md
Outdated
#### 2. Decoupled | ||
|
||
In the second option, you define the event manually in any service, but link it to the respective cloud event type via `@topic`. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Event Broker enforces the [cloudevents](https://cloudevents.io/) standard and `@topic` refers to the cloudevents `type` property. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
event manually in any service
Not true, it must be in an external service. Otherwise it's considered to be an event the app publishes.
Add an _Integration Dependency_ using the _Simplified Business Eventing Template_ for _Event Type_ `sap.eee.iwxbe.testproducer.v1.Event.Created.v1` and _Publishing System Namespace_ `sap.s4` as shown in the following screenshot. | ||
|
||
![Integration Dependency for Test Event](assets/event-broker-test-integraion-dependency.png) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2. Create Formation
missing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, formation will already exist (event broker docs)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
formation needs to be manually created
[Learn more about `cds.env` profiles](../../node.js/cds-env#profiles){.learn-more} | ||
|
||
::: tip Local Testing | ||
Since SAP Event Broker sends events via HTTP, local |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding a comment so that we don't miss the incomplete sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
didn't help 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sjvans Will you complete the sentence? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sjvans Will you complete the sentence? :)
i don't know what @David-Kunz wanted to say here ;) but yes, we'll take care of it
No description provided.