-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
[14.0] [IMP] base_rest: support multiple types #415
base: 14.0
Are you sure you want to change the base?
Conversation
Hi @lmignon, |
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.
@renda-dev Thank you for the improment. Can you fix the tests plz?
3df70f2
to
f50cc3c
Compare
Greetings, Is it possible to keep going with this PR anyway? @lmignon |
It's my understanding that #417 is required for this PR's tests to pass, is that correct? |
@renda-dev #417 is on its path to be merged. Once merged can you rebase your PR to make it 🟢 |
f50cc3c
to
52a8366
Compare
Done, thanks! @lmignon |
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.
LGTM (Code review only)
The main goal for this PR is to add multi-type support when generating swagger docs.
At the current state, even tho cerberus validator supports multi-type validation, swagger would raise an error without giving access to the api's docs.
To do so, I've also updated Swagger UI and OpenAPI to the LTS.
Depends on #417