Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature/requestBodies param support #152

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Feature/requestBodies param support #152

wants to merge 5 commits into from

Conversation

StasanTelnov
Copy link

This PR implements the requestBodies schemas (148).

  • Added requestBodies models generation
  • Added "name" param for every schema: containts original name, whithout suffixes and prefixes
  • Added 4 template params: modelRequestBodyPrefix, modelRequestBodySuffix, modelResponsePrefix, modelResponseSuffix
  • Updated template for example requestBodies and new params example
  • Updated readme with new prefix and suffix description

Stanislav Telnov added 3 commits April 9, 2019 17:41
Add requestBodies params
a4a2fbc 
Update schemas parsing
Refactor request bodies parsing
194c15b 
Added prefixes for request bodies models and responces models
Update template: added suffixes and prefixes for request and response…
7832e6a 
…s models

Fix typo in "response"
Update readme
@StasanTelnov
Copy link
Author

Hello, @yonaskolb
Test of my PR failed, because specification was updated. I dont checked it before create PR.
I can update specification or remove my changes in template file.
But if I delete the changes in the template file, the template will not show how reguestBodies can be used.
How do I do better in your opinion?

Stanislav added 2 commits May 16, 2019 09:48
Add safeEnum
8e89d03 
Remove never executed code
Fix response handler
6caf869
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@StasanTelnov