-
Notifications
You must be signed in to change notification settings - Fork 427
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
[JAVA] Implement #10176 (for 3.x) #667
base: master
Are you sure you want to change the base?
Conversation
@michaelkourlas - Solution looks neat. One of the API that I consume has 10 optional parameters and I had to set null for all of them. Can this be added to the |
I'd like the maintainers to confirm whether they're interested in merging this PR before I make any further additions. |
hey @michaelkourlas thanks for this PR, let me know once is ready to review it and merge it |
@HugoMario I've added support for this pattern for resttemplate as requested, so I believe this PR (as well as this one and this one) are ready to be reviewed. |
@HugoMario Just notifying you that this PR (as well as this one and this one) are ready to be reviewed. |
Implemented #10176 (from the swagger-codegen repo) for jersey1, jersey2, resttemplate, and okhttp-gson. Each API call now has up to two additional methods:
It was necessary to fix a bug where the body parameter did not appear in the requiredParams list.