-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
[BACK-2958] [BACK-2960] [BACK-3076] Add ability to execute clinic merge plans #154
base: master
Are you sure you want to change the base?
Conversation
b1f517f
to
79df27e
Compare
return err | ||
} | ||
if res == nil { | ||
return fmt.Errorf("patient list result not defined") |
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.
Is there a reason sometimes the error is defined in the errors
package and sometimes defined inline?
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 errors in the errors
package ought to wrapped so when they bubble up to the API layer we return the correct http code. All other errors which bubble up with result in a 500 response, which is intended here.
[BACK-3076] Persist executable merge plans for audit logging purposes
d9e1343
to
9b3abd3
Compare
No description provided.