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

Ability to set permissions for aliases #2944

Open
kyle95wm opened this issue Jan 26, 2021 · 2 comments
Open

Ability to set permissions for aliases #2944

kyle95wm opened this issue Jan 26, 2021 · 2 comments

Comments

@kyle95wm
Copy link

Is your feature request related to a problem? Please describe.
Let's say there may be aliases that run commands that aren't meant for lower tier staff members such as the support team but are intended for moderators. The problem is, an alias is just a shortcut to run a string of commands, and I'm pretty sure those bypass permissions.

Describe the solution you'd like
It would be nice to set permissions of some kind (like how you can define certain commands to belong to a certain level) that allows for an alias to be defined to a specific permissions level. For example, a level 3 alias would send a message to the user to let them know that they've abused modmail, block them for 3 days, then close their thread silently. A supporter or lower will obviously not be able to run this, if permissions were defined at the alias level.

Describe alternatives you've considered
Short of running each command separately (which respects permission levels) it's not ideal if you want to do it quickly.

Who will this benefit
This will benefit anyone who may wish to have more granular control over who can run what alias

Additional context
n/a

@StephenDaDev
Copy link
Member

A general comment is, they don't bypass permissions, they use the permission the command they are imitating has, for example, eval is level 5, so level 5 is also applied if you had ?e as ?eval. And from what I've heard, permissions in aliases are not likely to come, the amount of effort and reworking isn't worth it in the long run.

@fourjr
Copy link
Collaborator

fourjr commented Jan 29, 2021

As @StephenDaDev said, this already exists in some form. Perms of the command to be executed are already taken into account. The ability for custom alias permlevels will remain as a feature request though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants