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

Notify moderation room when users in protected rooms mention the bot (configurable) #553

Merged
merged 6 commits into from
Oct 25, 2024

Conversation

Half-Shot
Copy link
Contributor

Fixes #89

This works by checking to see if the bot has been mentioned in any of the protected rooms when a message comes in, and then will ignore any future mentions for the next 8 minutes (giving administrators a chance to deal with the offense, and to reduce spam).

This is off-by-default, as it's enough of a behaviour change that I think administrators should opt into this explicitly.

@Half-Shot Half-Shot requested a review from a team as a code owner October 22, 2024 09:53
Copy link
Contributor

@H-Shay H-Shay left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! LGTM. It'd be nice to verify in the test that subsequent mentions are ignored but I don't think that's necessary to merge :)

@Half-Shot Half-Shot merged commit 0f91bce into main Oct 25, 2024
5 checks passed
@Half-Shot Half-Shot deleted the hs/notify-on-mention branch October 25, 2024 16:10
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.

Notify admins when bot is @-mentioned
2 participants