Skip to content

Support 202 HTTP responses#116

Merged
bboe merged 1 commit intopraw-dev:masterfrom
MaybeNetwork:accept-202
Jun 10, 2021
Merged

Support 202 HTTP responses#116
bboe merged 1 commit intopraw-dev:masterfrom
MaybeNetwork:accept-202

Conversation

@MaybeNetwork
Copy link
Copy Markdown
Contributor

Feature Summary and Justification

Adds HTTP response status code 202 to the list of successful status codes. The endpoint api/read_all_messages returns this response.

References

Copy link
Copy Markdown
Member

@bboe bboe left a comment

Choose a reason for hiding this comment

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

Minor change necessary if I end up merging this one first -- it might need to changed on the other PR if you want me to merge that one first.

Comment thread CHANGES.rst Outdated
prawcore follows `semantic versioning <http://semver.org/>`_.

Unreleased
------------------
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For RST this line should only be as long as the characters above it.

Copy link
Copy Markdown
Contributor Author

@MaybeNetwork MaybeNetwork Jun 10, 2021

Choose a reason for hiding this comment

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

Ok, I'll fix that on this one and the other

@bboe bboe merged commit 0654440 into praw-dev:master Jun 10, 2021
@bboe
Copy link
Copy Markdown
Member

bboe commented Jun 10, 2021

Thanks for the PR! 🎆

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.

2 participants