Don't count bot mentions as ghost pings by userbyte · Pull Request #107 · allthingslinux/tux
Navigation Menu
{{ message }}
allthingslinux / tux Public
- Notifications You must be signed in to change notification settings
- Fork 42
Merged
electron271 merged 2 commits intoallthingslinux:mainfrom
Mar 31, 2024Merged
Don't count bot mentions as ghost pings#107
electron271 merged 2 commits intoallthingslinux:mainfrom
Don't count bot mentions as ghost pings#107
electron271 merged 2 commits intoallthingslinux:mainfrom
Conversation
Copy link Copy Markdown
Contributor
userbyte
commented
Mar 31, 2024
userbyte
commented
If a user pings a bot and deletes the message, there is no need to trigger a ghost ping message because the mentioned user is a bot.
electron271
requested review from
electron271 and
kzndotsh
electron271
assigned
userbyte and
electron271
electron271
added
the
events
label
electron271 reviewed Mar 31, 2024
tux/cogs/utility/ghost_pings.py
Outdated
Show resolved
Hide resolved
tux/cogs/utility/ghost_pings.py Outdated Show resolved Hide resolved
userbyte
requested a review
from electron271
electron271 approved these changes Mar 31, 2024
Copy link Copy Markdown
Member
electron271
left a comment
electron271
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
electron271
merged commit
31220ff
into
allthingslinux:main
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants