Skip to content

Conversation

@Fealtous
Copy link
Contributor

Hardcoded "Patreons" kinda ugly. Could replace with just Role of target > Role of initiator but that has its own issues.
Purpose is to allow s!lent commands to be truly directed at a user. Should hopefully decrease some confusion when multiple people are being assisted at once. Bot will not ping if no user is replied to. Bot will not ping any of the Hoisted roles except Patreons.

@RealMangorage
Copy link
Member

Why does it need to ping?

@Fealtous
Copy link
Contributor Author

Fealtous commented Mar 20, 2025

It doesn't necessarily have to ping, but it helps make it more obvious a command was used with regards to a specific person. Some people don't seem to understand when a command is directed towards them. It's particularly difficult to deal with when there's several people asking for help at the same time.

If you'd like that it just replies without the ping that's an easy enough change.

@RealMangorage
Copy link
Member

Ah okay, can you provide me some examples I can look at? So I can see how It looks. Thanks

@Fealtous
Copy link
Contributor Author

image
First is example of pinging when a user is of the "can be pinged" list, lower is if not. Hopefully this actually attaches.

@RealMangorage
Copy link
Member

Also is there a reason why we only ping the Patreon users only?

@Fealtous
Copy link
Contributor Author

It won't ping hoisted users, however it is possible that a patron may need assistance and although they are hoisted, they aren't staff members, so I think it's fine to ping them.

@RealMangorage
Copy link
Member

Gotcha

@Fealtous
Copy link
Contributor Author

image
I realize my previous picture might be a little unclear so here's a fantastic ms paint representation of where the replies are directed.

Copy link
Member

@RealMangorage RealMangorage left a comment

Choose a reason for hiding this comment

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

Looks good

@RealMangorage RealMangorage merged commit 628564f into MangoRageBot:master Mar 20, 2025
1 check passed
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