-
Notifications
You must be signed in to change notification settings - Fork 476
Allow for entity documentation to support an array of entities #71
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
Conversation
@@ -72,5 +85,50 @@ def app; ModelsApi; end | |||
} | |||
} | |||
end | |||
|
|||
it "should include multiple response_classes when specified" do |
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.
This test should be a lot more explicit, doing a JSON.parse
and looking specifically for the things affected by multiple response_classes
.
This needs a rebase, please and see my comment on the test above. Thank you. Sorry this took so long. A few people are taking over the project and things are going to move forward now. |
What about this pull request? |
Look to ^^^ from @matt61 (or anyone else who wants to finish this). |
@matt61 +1 |
I think https://github.com/tim-vandecasteele/grape-swagger/pull/217 closes this, but there's a lot more code here doing all kinds of things. Please reopen/comment if I am missing something. |
This allows for using the $ref element in the swagger spec to display nested relationships.