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

DES-167: add banned phrases #304

Merged
merged 12 commits into from
Apr 1, 2024
Merged

DES-167: add banned phrases #304

merged 12 commits into from
Apr 1, 2024

Conversation

vyneer
Copy link
Contributor

@vyneer vyneer commented Aug 15, 2023

  • Adds native banned phrases support
  • Adds /addban and /deleteban commands to add and remove banned phrases
    - Adds /die command to mute yourself for 10 minutes and send a DEATH event in chat

Needs https://github.com/destinygg/website-private/pull/227 to work.

@vyneer vyneer added the enhancement New feature or request label Aug 15, 2023
@vyneer vyneer requested a review from 11k August 15, 2023 13:33
@ILiedAboutCake
Copy link
Collaborator

remove /die or make it no-op in chat

@vyneer
Copy link
Contributor Author

vyneer commented Aug 22, 2023

remove /die or make it no-op in chat

Wait, really? :( Is there a reason why? I thought it was a cute idea, people bitly themselves rn in protest/for fun sometimes.

(sorry for giga late reply)

Edit: removed it.

@11k
Copy link
Contributor

11k commented Aug 29, 2023

/die is essential so we'll add it in a separate PR, but it absolutely shouldn't have the same level of emphasis as a sub or dono. I think just an italicized status message like /me is fine. We'll also remove the "last words" behavior to keep things simple. The user can just utter their last words before using the command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants