-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix project access error message #379
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.
Nice. Collapsing multiple keys for unique errors will give a much better UX!
…elabDataCentre/dds_cli into fix-internal-server-error
I know that there's some duplication, will change this at some point, now I just need it to give an understandable message. |
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.
Nice. One minor thing that I don't understand, but nothing that would preclude approval.
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.
Some other non-related bugs found during testing, but the code covered here seems to work as intended.
Print out which projects a user could not give access to and the errors. Should be used with ScilifelabDataCentre/dds_web#1045
Before submitting a PR to the
dev
branch:dev
branchAdditional checks before submitting a PR to the
master
branch:setup.py
(?)