-
Notifications
You must be signed in to change notification settings - Fork 37
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
Refactor RequestSchema and ResponseSchema into a single Schema #161
Comments
I do like that having two classes makes the intention of derived classes clear. I hardly think that outweighs the negatives you've laid out though. |
Reading over the comments on #152, I'd like to propose that we name this unified single Schema something like I think that might make it a bit clearer when explaining that |
I like the name, I will work on the implementation |
RequestSchema
is only needed in v2 to reject additional fieldsResponseSchema
is currently not playing too well with IDE (pycharm cough) because its missing a constructor with the kwargs (see ResponseSchema is missing constructor parameters #137)For those reasons I think we should refactor that into a single class that we use in all the tutorials and doc.
The text was updated successfully, but these errors were encountered: