Skip to content

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

Closed
wants to merge 9 commits into from

Conversation

matt61
Copy link

@matt61 matt61 commented Nov 15, 2013

This allows for using the $ref element in the swagger spec to display nested relationships.

@@ -72,5 +85,50 @@ def app; ModelsApi; end
}
}
end

it "should include multiple response_classes when specified" do
Copy link
Member

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.

@dblock
Copy link
Member

dblock commented Jul 14, 2014

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.

@onemanstartup
Copy link
Contributor

What about this pull request?

@dblock
Copy link
Member

dblock commented Feb 9, 2015

Look to ^^^ from @matt61 (or anyone else who wants to finish this).

@swistaczek
Copy link
Contributor

@dblock
Copy link
Member

dblock commented Feb 20, 2015

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.

@dblock dblock closed this Feb 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants