Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Allow a wordpress api request to specify certain status codes it want... #12

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

Merged
d3v-null merged 2 commits into d3v-null:master from december1981:master
May 5, 2019
Merged

Allow a wordpress api request to specify certain status codes it want... #12

d3v-null merged 2 commits into d3v-null:master from december1981:master
May 5, 2019

Conversation

Copy link

@december1981 december1981 commented Mar 16, 2019

...s to allow/handle in response.

There are many cases where I might decide eg a 400 response is helpful - I have an import script for which it's more convenient to try and post something, and handle a 400 - telling me the item is already there, from a previous run of the script... rather than querying if it's there then posting only if it's not (which may be on average a slower process especially on the first run.)

Copy link
Author

If you approve this modification, I'll update the readme if you like, as part of this.

Copy link

codecov bot commented May 5, 2019
edited
Loading

Codecov Report

Merging #12 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@ Coverage Diff @@
## master #12 +/- ##
==========================================
+ Coverage 85.96% 85.97% +0.01% 
==========================================
 Files 5 5 
 Lines 755 756 +1 
==========================================
+ Hits 649 650 +1 
 Misses 106 106
Impacted Files Coverage Δ
wordpress/api.py 72.58% <100%> (+0.22%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60fec3f...1326d89. Read the comment docs.

Copy link
Owner

d3v-null commented May 5, 2019

Hey there, thanks for the PR, especially since it includes tests!

december1981 reacted with thumbs up emoji

@d3v-null d3v-null merged commit bfeb983 into d3v-null:master May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /