Skip to content

Change DM users get when warned#2696

Merged
Xithrius merged 2 commits into
mainfrom
swfarnsworth/warning-embed
Aug 5, 2023
Merged

Change DM users get when warned#2696
Xithrius merged 2 commits into
mainfrom
swfarnsworth/warning-embed

Conversation

@swfarnsworth
Copy link
Copy Markdown
Contributor

Create separate message for warnings without duration, expiry, or "to appeal this infraction ..."

Since warnings involve no change in permissions, the DM that users receive said "Duration: Permanent \ Expires: Never", which some interpreted to mean that we hold a permanent grudge. This is not intended, so this commit creates a separate template that omits those fields. Some users also thought they could appeal a warning to have it removed from the record, but this is not the case, so this commit creates a separate footer that removes "or appeal" from the instruction to contact the ModMail bot.

Showing the new message received when warned:
image

Showing the message received when muted (which has not changed):
image

(The ModMail bot is not in the user cache for the dummy account that I muted.)

… appeal this infraction ...".

Since warnings involve no change in permissions, the DM that users receive said "Duration: Permanent \\ Expires: Never", which some interpreted to mean that we hold a permanent grudge. This is not intended, so this commit creates a separate template that omits those fields. Some users also thought they could appeal a warning to have it removed from the record, but this is not the case, so this commit creates a separate footer that removes "or appeal" from the instruction to contact the ModMail bot.
Copy link
Copy Markdown
Contributor

@Xithrius Xithrius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good, thank you

@Xithrius Xithrius enabled auto-merge (squash) August 5, 2023 04:57
@Xithrius Xithrius merged commit e56cd0c into main Aug 5, 2023
@Xithrius Xithrius deleted the swfarnsworth/warning-embed branch August 5, 2023 04:58
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.

3 participants