-
Notifications
You must be signed in to change notification settings - Fork 27
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
✨ Frontend: Ask for confirmation before demoting an Organization to Viewer #3961
✨ Frontend: Ask for confirmation before demoting an Organization to Viewer #3961
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.
Thanks!
I'd change the message to "Demoting to Viewer will remove write access to all the members of the Organization. Are you sure?"
And change the confirmation button string to "Yes" (instead of "Demote").
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.
👍
Code Climate has analyzed commit 1164c3d and detected 0 issues on this pull request. View more on Code Climate. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
What do these changes do?
This PR makes the frontend ask for confirmation when a user tries to Demote a Organization's role in a study/template collaboration.
This confirmation can be disabled in the preferences.
Related issue/s
closes ITISFoundation/osparc-issues#901
How to test
Checklist