-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Edit permissions in bulk by using cog names GH#2900 #3202
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.
Took a quick peek. Looks good. 👍
Got a couple of suggestions there. Thoughts?
Not sure about the .update_perms
tho 🤔.
Update:
One more thing, the group name is case sensitive and sometimes it could be multiple words too, if I remember correctly.
Yeah group name is case sensitive, thats why I take it and put it in first letter capitalized, this should work for all inbuilt cogs. For plugins however, they will need to enter the exact class name if it isnt named as first letter capitalised. And no, class names cant be more than one word so I dont think the group name can be more than one word! |
Yes, class name cannot have multiple words. But cog name could have multiple words (by specifying the |
Co-authored-by: Jerrie <[email protected]>
Co-authored-by: Jerrie <[email protected]>
Cant they just use "cog name" 😕 |
Resolves #2900, a 2 year old feature request.
Have tested this out properly and ironed out all bugs to my understanding. Attached a few screenshots to help understand better.