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] Periodic Permissions Checking #83

Open
logannc opened this issue Jan 27, 2021 · 3 comments
Open

[Feature Request] Periodic Permissions Checking #83

logannc opened this issue Jan 27, 2021 · 3 comments

Comments

@logannc
Copy link

logannc commented Jan 27, 2021

Store the state of permissions available to Viav for a given Guild. If they change, verify that Viav can still operate correctly (i.e., we don't care about all permission changes). If it changes in a way that affects the ability for Viav to operate, notify the guild administrators somehow.

Example:

If Manage Channels or Manage Roles is removed, ping the admin to alert them that this will break viav.
Vice versa, if those permissions are added, alert the admin to the restored functionality.

Would need some way to configure where notifications are sent and a way to mute/ignore/etc.

Implicitly, it also means Viav needs to be able to detect these changes and know what it means for itself.

@BrandonDyer64
Copy link
Contributor

I've thought about implementing something like this. Viav unfortunately needs almost every permission to actually work. I'm sure there's a combination of perms that would allow it to work in a minimal sense, but in my testing I could only get it to function consistently when I gave it admin.

@logannc
Copy link
Author

logannc commented Jan 28, 2021

do you have a short list of places where you ran into permission problems? i can find them on my own, but it would take a while lol

@BrandonDyer64
Copy link
Contributor

Pretty much just in the channel creation and deletion scripts. It wasn't able to follow through without admin.
Starting in this fairly long function.

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