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

feat(Rest): show the data that is sent to Discord when an errors occurs #5701

Merged
merged 3 commits into from Jun 3, 2021

Conversation

vladfrangu
Copy link
Member

Please describe the changes this PR makes and why it should be merged:

Now you know what Discord means by your embed being wrong... or that color being invalid...Or just in general what we sent that caused the error you received

Status and versioning classification:

  • This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
    • Above is a stretch, because this is internals

Now you know what Discord means by your embed being wrong...
src/rest/DiscordAPIError.js Outdated Show resolved Hide resolved
@vladfrangu vladfrangu force-pushed the feat/show-sent-data-in-rest-errors branch from 72f4432 to 5c09517 Compare May 29, 2021 17:38
@vladfrangu vladfrangu force-pushed the feat/show-sent-data-in-rest-errors branch from 5c09517 to 361f338 Compare May 29, 2021 17:52
src/structures/APIMessage.js Outdated Show resolved Hide resolved
src/structures/APIMessage.js Outdated Show resolved Hide resolved
@iCrawl iCrawl merged commit ef92339 into master Jun 3, 2021
@iCrawl iCrawl deleted the feat/show-sent-data-in-rest-errors branch June 3, 2021 08:35
@iCrawl iCrawl added this to the Version 13 milestone Jun 27, 2021
ckohen added a commit to ckohen/discord.js-modules that referenced this pull request Oct 11, 2021
Ported from discordjs/discord.js#5701

Co-authored-by: Vlad Frangu <kingdgrizzle@gmail.com>
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.

None yet

6 participants