You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
The cog does not check what roles have been changed in the audit log entry which causes it to report incorrect "Updated by" and "Reason" in the logs it reports.
To Reproduce
Have multiple roles applied by different bots at the same time and hope the cog gets it mixed up? There are screenshots below of how that happened in my case.
Expected behavior
The audit log entry should be checked for the changed roles.
Screenshots
// What's up with these sections? 🤔 I'm using up-to-date Red and ExtendedModLog but I don't think you need to know my browser or smartphone :)
Desktop (please complete the following information):
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]
Smartphone (please complete the following information):
Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]
Additional context
Here is how the above audit log entries look in d.py
(reprs of target and user simplified to just username in order to save space but they are regular discord.Member objects):
Describe the bug
The cog does not check what roles have been changed in the audit log entry which causes it to report incorrect "Updated by" and "Reason" in the logs it reports.
To Reproduce
Have multiple roles applied by different bots at the same time and hope the cog gets it mixed up? There are screenshots below of how that happened in my case.
Expected behavior
The audit log entry should be checked for the changed roles.
Screenshots
// What's up with these sections? 🤔 I'm using up-to-date Red and ExtendedModLog but I don't think you need to know my browser or smartphone :)
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
Here is how the above audit log entries look in d.py
(reprs of target and user simplified to just username in order to save space but they are regular discord.Member objects):
Additional notes
This is probably not the only event that should be checked for more than just the target, see the d.py documentation for more information:
https://discordpy.readthedocs.io/en/stable/api.html#discord.AuditLogAction
The text was updated successfully, but these errors were encountered: