-
Notifications
You must be signed in to change notification settings - Fork 31
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
Feature/check for 404 #262
Conversation
…there is a communication error with the target oracle endpoint.
…UNICATION_ERROR` && `NOTFOUND.CODE` to be `DESTINATION_FSP_ERROR` instead of `PARTY_NOT_FOUND`
# Conflicts: # src/models/oracle/facade.js # test/unit/handlers/participants/{Type}/{ID}.test.js # test/unit/handlers/participants/{Type}/{ID}/{SubId}.test.js # test/unit/handlers/parties/{Type}/{ID}.test.js # test/unit/handlers/parties/{Type}/{ID}/{SubId}.test.js
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.
Please updated version in package.json for re-release
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.
Approved pending Rajiv's suggested changes. Thanks!
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.
Happy please await tests
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.
Happy!
Changed the returned error code on the Mowali edge case of
DESTINATION_COMMUNICATION_ERROR
&NOTFOUND.CODE
to beDESTINATION_FSP_ERROR
instead ofPARTY_NOT_FOUND