-
Notifications
You must be signed in to change notification settings - Fork 0
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
207 authorization bugs #220
Conversation
…of manual status checking
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.
Just a couple comments.
I'm not familiar enough with the workflow, so I assume the statuses removed from USER_CAN_EDIT_AWARD_BORROWER_DATA
and the new guards to upload_compliance, download_application and update_apps_send_notifications are correct.
Yes, in the past this same list was used for listing applications, for example, now it is only used for the editing endpoints so we only need the actual editable status. |
Co-authored-by: James McKinney <26463+jpmckinney@users.noreply.github.com>
…credere-backend into 207-authorization-bugs
Pull Request Test Coverage Report for Build 7023047994
💛 - Coveralls |
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.
Just double-check my merge conflict resolution: 8be9260
Closes #207