-
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
fix(ui): fix New Submission
button display
#794
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.
Awesome work!
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.
This looks good man. Glad we consolidated it to not use the userContext anymore
Code Climate has analyzed commit c8afd46 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 59.6% (50% is the threshold). This pull request will bring the total coverage in the repository to 62.1% (0.2% change). View more on Code Climate. |
🎉 This PR is included in version 1.5.0-val.87 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Purpose
New Submission
button was observed to show up for users who didn't have permission to create new submissionsLinked Issues to Close
https://qmacbis.atlassian.net/browse/OY2-30386
Approach
Seems like the
isCms
property is missing from the user object received from the API. @13bfrancis, is this an intentional change or a bug? If the latter, we should remove all reliance onisCms
since it's used extensively across the app.