-
Notifications
You must be signed in to change notification settings - Fork 5
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
Simple Order and Shipment Workflows #2
Conversation
@@ -0,0 +1,34 @@ | |||
package activities |
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.
@cretz Any preferences on the question of whether notifications like these should be defined by the source (pubsub-style) or by the receiver (which I would call master/slave style, which I know is now considered very politically incorrect but I don't know any good equivalent).
I tend to prefer the latter, which would imply (here, I think) that the OP App defines the information it needs and treats other systems as auxiliary.
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.
Let's discuss on Slack.
Co-authored-by: Paul Nordstrom <85532575+paulnpdev@users.noreply.github.com>
No description provided.