-
-
Notifications
You must be signed in to change notification settings - Fork 119
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
GUI Permission Editor #1139
Comments
I've made a draft GUI editor. This involves moving the permissions to the database instead of being hardcoded or in a config file, which also requires moving UserClass code to an extension (so the database can load first). There are a few code issues marked by |
I wonder why this was in a config file in the first place, since being in the database certainly feels like a better idea... my gut is telling me that there was a reason, but I can't remember it 😅 If it's in the database in your branch, and everything is working fine there, maybe the reason disappeared at some point over the past few years... |
It could have been because the database code loads after core code. Everything seems to be working alright so hopefully the reason disappeared :) |
Making a proper feature request for this suggestion:
Describe the solution you'd like
A config page where the permissions of each role can be configured by administrators using checkboxes.
Special roles like admin should have their permissions visible with checkboxes greyed out. If a role has a parent role, this should be added using a dropdown box, so the user understands which parent role must be edited to remove an inherited permission, or can change the parent role.
In the future, this should be able to add new roles or delete unused roles.
The text was updated successfully, but these errors were encountered: