-
Notifications
You must be signed in to change notification settings - Fork 2
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: introduce @validate_body_root_model #423
Conversation
Pull Request Test Coverage Report for Build 11066173668Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
You can access the deployment of this PR at https://renku-ci-ds-423.dev.renku.ch |
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.
thanks for this!
I have two bugfix PRs open in sanic (one addresses this issue) for over a month now, without any reply. So don't have too high hopes of them fixing this any time soon on their end...
Tearing down the temporary RenkuLab deplyoment for this PR. |
Add
@validate_body_root_model
which is a drop-in replacement for Sanic's@validate
when validating aRootModel
or a model deriving from it.The decorator should be replaced by
@validate
once sanic-org/sanic-ext#198 is fixed./deploy