-
-
Notifications
You must be signed in to change notification settings - Fork 751
Allow role mentions in specific areas #1038
Copy link
Copy link
Closed
Labels
a: utilityRelated to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)Related to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)p: 1 - highHigh PriorityHigh Prioritys: planningDiscussing detailsDiscussing detailst: bugSomething isn't workingSomething isn't working
Metadata
Metadata
Assignees
Labels
a: utilityRelated to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)Related to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)p: 1 - highHigh PriorityHigh Prioritys: planningDiscussing detailsDiscussing detailst: bugSomething isn't workingSomething isn't working
Type
Fields
Give feedbackNo fields configured for issues without a type.
Description
Since we do not allow the bot to mention roles anymore (b1c0177) except for moderators+ (36330b1) after the incident, this has created a new issue where the bot cannot ping roles where it is actually needed in certain places. This needs to be fixed.
Some of these places include:
@everyone)@everyone)@Helpers)@PyDis Core Developers)There may be some other areas I missed, so please look over the codebase carefully and explicitly allow role mentions where needed.
Implementation
We can use
discord.AllowedMentionswhen sending messages where non-whitelisted role mentions are needed, which will override the global setting. This object can be passed using theallowed_mentionskwarg.