Skip to content
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

Only let users unassign roles they are allowed to assign #1691

Merged
merged 4 commits into from
Sep 7, 2023

Conversation

Masterjun3
Copy link
Collaborator

Resolves #1690 .
It also supports neat disabled attributes for the client view.
image

@Masterjun3 Masterjun3 requested a review from adelikat September 6, 2023 19:34
@vadosnaprimer
Copy link
Collaborator

I suggest making text for disabled list items gray, like on disabled buttons.

@Masterjun3
Copy link
Collaborator Author

image

@vadosnaprimer
Copy link
Collaborator

Perfect. Now I can see if I'm able to set the cleared ones or not.

@adelikat adelikat merged commit 09e64df into main Sep 7, 2023
1 check passed
@adelikat adelikat deleted the userrole-edit-fix branch September 7, 2023 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Role editing grants unexpected privileges
3 participants