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

allow staff to be limited by CA #91

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jakjakob
Copy link

@jakjakob jakjakob commented Jul 3, 2024

the reason of this is that by setting CA to also affect Rank 1 it will be overwritten by the check (I removed). This is problematic because it allows to set that up in the setup process, but at the end does not work (without knowing of the person that set that up, unless tested). In my setup I want to keep it activated also for mods, as it just deletes messages and so does not risk to punish in any way the staff.
Even though you may have done that to avoid a staff member to accidentally get banned, so there might be needed a warning message to be sent before configuring it that way (I think personally there is no need for that, as for example the Discord automod also affects mods (except admins and manage server perms) on default but does not make aware of this in any explicit way);
and/or (I have't looked at the other modules yet) to make it the same everywhere.
If you want I can take a look into it, just let me know

the reason of this is that by setting CA to also affect Rank 1 it will be overwritten by check. This is problematic because it allows to set that up in the config, but at the end does not work.
@jakjakob jakjakob marked this pull request as draft July 22, 2024 22:27
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

Successfully merging this pull request may close these issues.

1 participant