Conversation
Tomoli75
left a comment
Tomoli75
left a comment
There was a problem hiding this comment.
Would change the channel config key to something related to spam bots, as otherwise could be confused with storing deleted messages - lgtm otherwise
| @Cog.listener() | ||
| async def on_message(self, message: discord.Message): | ||
| joined_recently = message.author.joined_at > datetime.now() - timedelta(days=7) | ||
| contains_everyone = '@everyone' in message.content |
There was a problem hiding this comment.
Does the text in the message for an @ everyone actually come out as @ everyone? I know pings of roles/people are <@12356789>?
There was a problem hiding this comment.
yes it does, as if they don't have the permission it just formats to @ everyone, in terms of actual @ everyone pings, it does the same
| ) | ||
|
|
||
| await message.delete() | ||
| await channel.send(f'<@&{CONFIG.UWCS_EXEC_ROLE_IDS[1]}>', embed=embed) |
There was a problem hiding this comment.
Why take the 2nd role? That feels unintuitive at best - create a separate config param if you want it to be a specific role (that isn't just exec)
There was a problem hiding this comment.
i don't know? ask whoever put the exec role as the second role in the list...
| from config import CONFIG | ||
|
|
||
|
|
||
| class Database(Cog): |
There was a problem hiding this comment.
Why is the cog called database?
There was a problem hiding this comment.
fuck, that's actually my bad
3 participants
As above