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.

Sign up for GitHub

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

Jump to bottom

Warning infractions in mod channels should fail if the user has DMs off #1352

Open
swfarnsworth opened this issue Jan 15, 2021 · 3 comments · May be fixed by #3073
Open

Warning infractions in mod channels should fail if the user has DMs off #1352

swfarnsworth opened this issue Jan 15, 2021 · 3 comments · May be fixed by #3073
Assignees
@vivekashok1221
Labels
a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 3 - low Low Priority t: feature New feature or request

Comments

@swfarnsworth
Copy link
Member

Currently, if a moderator issues a warning to a user in a channel that that user can't see (usually a moderator-only channel), and that user happens to not receive DMs from non-friends, the user won't know that they've been warned and the point of the warning is effectively missed.

In this circumstance, the warning should not be entered into the database and the moderator should be informed that the warning has failed.

This change should not be made until @scragly's proposed changes to the infraction architecture are complete. See #1189.
@swfarnsworth swfarnsworth added a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 3 - low Low Priority labels Jan 15, 2021
@Xithrius Xithrius added s: planning Discussing details t: feature New feature or request labels Jan 15, 2021
@Xithrius
Copy link
Member

So if the warning or other infraction type fails, we can redo it in a public channel?

@swfarnsworth
Copy link
Member Author

@Xithrius I'm only proposing a change to warning infractions, since the warning message is the point of that infraction.

I don't think there's currently a solution for permission-reducing infractions (mutes, bans, etc.). If someone has DMs turned off, I suppose part of the trade-off they're making is that they lose the opportunity to know without inquiring why their permissions were reduced.

@Xithrius
Copy link
Member

Understandable. Good idea you got here, I too think that this should be implemented once #1189 is completed.

@Xithrius Xithrius added s: stalled Something is blocking further progress and removed s: planning Discussing details labels Feb 3, 2021
@vivekashok1221 vivekashok1221 self-assigned this May 24, 2024
@vivekashok1221 vivekashok1221 linked a pull request May 24, 2024 that will close this issue
Open
@vivekashok1221 vivekashok1221 removed the s: stalled Something is blocking further progress label May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vivekashok1221 vivekashok1221

Labels
a: moderation Related to community moderation functionality: (moderation, defcon, verification) p: 3 - low Low Priority t: feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make !warn fail if DMing the user is unsuccessful python-discord/bot
3 participants
@Xithrius @swfarnsworth @vivekashok1221