8000 Allow a wordpress api request to specify certain status codes it want… by december1981 · Pull Request #12 · d3v-null/wp-api-python · GitHub
[go: up one dir, main page]

Skip to content

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
merged 2 commits into from
May 5, 2019
Merged

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

merged 2 commits into from
May 5, 2019

Conversation

december1981
Copy link

…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.)

@december1981
Copy link
Author < 8000 /span>

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

@codecov
Copy link
codecov bot commented May 5, 2019

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.

@d3v-null
Copy link
Owner
d3v-null commented May 5, 2019

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

@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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0