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.

Sign up for GitHub

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

Jump to bottom

chore(http): 50109 error, invalid JSON #1338

Merged
merged 2 commits into from
Dec 18, 2021
Merged

chore(http): 50109 error, invalid JSON #1338

merged 2 commits into from
Dec 18, 2021

Conversation

vilgotf
Copy link
Member

@vilgotf vilgotf commented Dec 17, 2021

@vilgotf vilgotf added c-http Affects the http crate d-api Change related to Discord's API. t-chore labels Dec 17, 2021
Erk-
Erk- requested changes Dec 17, 2021
View reviewed changes
Copy link
Member

@Erk- Erk- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code should be added to test_variants in api_error.rs

7596ff
7596ff previously approved these changes Dec 17, 2021
View reviewed changes
@7596ff 7596ff merged commit cd949b1 into twilight-rs:main Dec 18, 2021
@vilgotf vilgotf deleted the chore/http/50109-error branch December 19, 2021 00:15
7596ff added a commit that referenced this pull request Dec 24, 2021
@7596ff
release(http): bump to 0.8.2
99ca081 
Additions

Add the API error code `50109 RequestInvalidJson` ([#1338] -
[@vilgotf]).

[#1326]: #1326
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Erk- Erk- Erk- approved these changes

@7596ff 7596ff 7596ff approved these changes

Assignees
No one assigned
Labels
c-http Affects the http crate d-api Change related to Discord's API.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vilgotf @7596ff @Erk-