-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat!: added outbound bulk api #320
Conversation
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.
The PATCH need to change to a PUT.
Sam raised the point that only Mojaloop can raise a Patch notification, and Patch should be a notification and not change or affect anything at the switch.
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.
It looks good to me based on the latest design.
However, is this API definition not generated from API snippets?
@shashi165 - can you also please change the version to 2.0.0 - this being a major change because of new resource addition |
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.
+1
BREAKING CHANGE:
The bulk transfers api has been re-defined, hence old bulkTransfers requests won't work anymore