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

[Feature Request] Rely on create/manage_events for /gcreate command permission #247

Open
alsoGAMER opened this issue Jun 22, 2023 · 5 comments

Comments

@alsoGAMER
Copy link

As also discussed in the Discord server with one support person (Naman), i think shifting away from manage_guild for some commands (like /gcreate) would benefit large communities (like mine) in helping not granting such dangerous permission to people in charge to only run giveaways/events in said community.

I hope you'll find the time to implement this or provide input on why it's not ideal to do it.

@jagrosh
Copy link
Owner

jagrosh commented Jun 22, 2023

That permission isn't required, it's just the default (if you don't make any overrides). If you want to allow a user or role to use a command, set that in the Integrations tab (make an override for each command you want to allow a user or role to use).

@alsoGAMER
Copy link
Author

If you want to allow a user or role to use a command, set that in the Integrations tab (make an override for each command you want to allow a user or role to use).

This i already know, what i'm saying tho is a change to default one.

@jagrosh
Copy link
Owner

jagrosh commented Jun 22, 2023

I guess I'm confused then, I feel like the default permission to use all the bot's commands should match the permission needed to be able to add the bot to the server in the first place. I'm also unsure how changing the default helps the aforementioned issue (granting a dangerous permission), since you can give someone access to run giveaways without giving them that permission.

@alsoGAMER
Copy link
Author

Lets say a user only has a role with only manage events, and that user runs the events aspect of a community, wouldn't be easier for admins to just grant that role without having to set specific overrides?

It also could allow for more granular access to commands, like

  • only giving settings access to users with manage server
  • while letting users with manage events run commands like gstart or redraw, etc

@jagrosh
Copy link
Owner

jagrosh commented Jun 22, 2023

If a server is looking for granular access, they are going to be making overrides. Everything you've described there is already possible via the Integrations tab.

As for baseline (default) functionality, I feel like it would be confusing for someone to be able to add the bot (clearly they are a manager of the community) but then be unable to use its commands (ie, if they have Manage Server but not Manage Events).

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

No branches or pull requests

2 participants