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

[Mutes] Update new channels #5116

Open
wants to merge 3 commits into
base: V3/develop
Choose a base branch
from

Conversation

laggron42
Copy link
Contributor

This adds a toggle under [p]muteset autoupdate for updating the permissions of new channels for the mute role.

Resolves #5111

@laggron42 laggron42 requested a review from TrustyJAID as a code owner June 3, 2021 08:49
@github-actions github-actions bot added Category: Docs - Other This is related to documentation that doesn't have its dedicated label. Category: Cogs - Mutes This is related to the Mutes cog. labels Jun 3, 2021
@TrustyJAID
Copy link
Member

I don't know if this is appropriate to handle considering channels are often auto-synced with channel categories when created and this can potentially break that sync.

@Jackenmen
Copy link
Member

Perhaps we could only handle it if it's not already in sync with the category?

@TrustyJAID
Copy link
Member

The thing is if it's not synced then the server mod who created the channel will potentially not want this messing with things and it should be fully controlled by the mod creating the channel. This setting could end up with a lot of confusion but checking for sync first as long as the cache is up-to-date in time for the bot to know about when the channel is created should suffice.

@Jackenmen Jackenmen added the Type: Feature New feature or request. label Jun 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Cogs - Mutes This is related to the Mutes cog. Category: Docs - Other This is related to documentation that doesn't have its dedicated label. Type: Feature New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Muted role for every new text channel
3 participants