-
Notifications
You must be signed in to change notification settings - Fork 6k
add @Valid annotion for bean-validation #5517
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
add @Valid annotion for bean-validation #5517
Conversation
@fisherliu thanks for the PR. Please also run ./bin/java-petstore-all.sh to update the Java Petstore samples so that the CIs can verify the test.
FYI. The CI tests with updated Petstore passed: https://travis-ci.org/swagger-api/swagger-codegen/builds/227529313
Let me know if you need help updating the Petstore samples for Java.
@wing328 ,thank you for your help. when will the change can be merged into master branch?it will to to version 2.2.3 or 2.3.0?
hi @wing328, what's the next step for the PR? I noticed it has passed the checks.
@fisherliu I've just merged it. Please update the Petstore samples next time so that the CIs can test the change.
when will the change can be merged into master branch?it will to to version 2.2.3 or 2.3.0?
Enhancements to master (2.2.3) will be sync'd to 2.3.0 as well.
TehBakker
commented
Jun 21, 2017
When will this be released?
Uh oh!
There was an error while loading. Please reload this page.
#5393:
https://github.com/cakemail/swagger-codegen/blob/07f066a20350302429c133f2febfcb58632aa56a/modules/swagger-codegen/src/main/resources/JavaSpring/beanValidation.mustache
the change below was added into JavaSpring language but it was not added into Java language.so I have created the PR to support @Valid for java language to implement#5498[JAVA] enhance beanvalidation with adding @Valid