-
Notifications
You must be signed in to change notification settings - Fork 46
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
Use integer response code values #766
Use integer response code values #766
Conversation
Build succeeded.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
can we get at least a single test case updated please?
85f018a
to
1d6110e
Compare
Build succeeded.
|
Done |
1d6110e
to
a4b6def
Compare
Build failed.
|
a4b6def
to
b17064c
Compare
what happened to zuul? /recheck |
Build succeeded.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you!
Build succeeded (gate pipeline).
|
Thanks to @Stranger6667 for reporting it in #368 (comment)
Basically we were sending and documenting the response status object instead of the integer code as required by RestX and the return value to a view should also have an integer code as status.
https://docs.python.org/3/library/http.html