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

Separate muteset senddm for mutes and unmutes #6262

Open
EternalllZM opened this issue Oct 28, 2023 · 3 comments
Open

Separate muteset senddm for mutes and unmutes #6262

EternalllZM opened this issue Oct 28, 2023 · 3 comments
Labels
Category: Cogs - Mutes This is related to the Mutes cog. Status: PRs Welcome No one is currently assigned to this issue, but we'd be grateful if anyone made a PR. Type: Enhancement Something meant to enhance existing Red features.

Comments

@EternalllZM
Copy link

What component of Red (cog, command, API) would you like to see improvements on?

mutes

Describe the enhancement you're suggesting.

Currently, [p]muteset senddm will send notifications for both mutes and unmutes. This proposes that the unmute dm be toggle-able separately from a mute.

Why? Notifying the user they have been unmuted reminds the user of why they were muted (big angry!), but gives them a first class pass to go complain about it in the server and cause issues immediately.

Without being notified, likely they will forget about it and return days later with a better perspective, also moving past it.

For non-behavioral reasons, this provides more control to the user to decide what notifications are sent to the user versus modlog only.

Anything else?

While the first point is an issue to be handled by moderation, I ask what the harm is in thwarting this behavior in the first place by not announcing unmutes to the muted.

@EternalllZM EternalllZM added the Type: Enhancement Something meant to enhance existing Red features. label Oct 28, 2023
@github-actions github-actions bot added the Status: Needs Triage This has not been labeled or discussed for handling yet. label Oct 28, 2023
@Jackenmen Jackenmen added Status: Accepted We want this Category: Cogs - Mutes This is related to the Mutes cog. and removed Status: Needs Triage This has not been labeled or discussed for handling yet. labels Oct 28, 2023
@turbo-jayce
Copy link

hey! i would like to work on this

@sxliang3093
Copy link

Could the project manager please assign the issue to me as i would love to work on this? Thank you!

@Flame442
Copy link
Member

@sxliang3093, You are free to create a PR for any issue labeled as "Accepted" or "PRs welcome". We don't generally assign people to issues outside of core contributors. When we don't have a great line of communication with a contributor and don't have a PR already created, we don't know if/when they decide to abandon working on the issue. If you create a PR, it will be linked to this issue to make it clear that it is in active development, and we can see the progress towards fixing this issue. Feel free to label your PR as a draft if you have not yet fully resolved this issue.

If you need any additional help contributing, feel free to reach out in our discord support server.

@Flame442 Flame442 added Status: In Progress There's a PR open to deal with this, or an org member is working on this internally and removed Status: Accepted We want this labels Apr 25, 2024
@Flame442 Flame442 added Status: PRs Welcome No one is currently assigned to this issue, but we'd be grateful if anyone made a PR. and removed Status: In Progress There's a PR open to deal with this, or an org member is working on this internally labels May 2, 2024
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. Status: PRs Welcome No one is currently assigned to this issue, but we'd be grateful if anyone made a PR. Type: Enhancement Something meant to enhance existing Red features.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants