Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete Message API change breaks bot features #738

Open
SebastiaanZ opened this issue Feb 4, 2020 · 0 comments
Open

Delete Message API change breaks bot features #738

SebastiaanZ opened this issue Feb 4, 2020 · 0 comments

Comments

@SebastiaanZ
Copy link
Member

@SebastiaanZ SebastiaanZ commented Feb 4, 2020

Recently, the deleted messages API received an update that introduced a new required field (attachments). Unfortunately, not every bot feature using this endpoint has been updated yet, causing the API to reject the requests.

One such feature that currently fails is bulk deletion by the Clean cog. Another one is part of the on_message_delete event listener in the modlog Cog. We should probably do a thorough check to see if there are more features that rely on this endpoint.

The fix should be straightforward, as we only need to send along an empty list for the attachments field to make it work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.