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.

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

Fix dynamic-html positioning of parameters #717

Merged
merged 1 commit into from
May 16, 2015

Conversation

ponelat
Copy link
Member

@ponelat ponelat commented May 4, 2015

Based on PR: #716
fixes #334
Re-factored the way parameters are viewed.

@wing328
Copy link
Contributor

wing328 commented May 14, 2015

@ponelat when you've time, I wonder if you can rebase this PR (and the other 2 PRs related to documentation as well) as it contains merge conflicts. Thanks!

@ponelat ponelat force-pushed the fix/dyn-html-pos-error branch from 6f495a5 to 3bc319d Compare May 15, 2015 07:29
@ponelat
Copy link
Member Author

ponelat commented May 15, 2015

@wing328 done. Also I've made it so they don't rely on other PRs... They are merge-able on my system. But just give a shout if they don't merge here...
The other two are here... #715 #716

@wing328
Copy link
Contributor

wing328 commented May 16, 2015

Thanks. I did a test and the parameter list looks good to me
screen shot 2015-05-16 at 10 57 17 pm

fehguy added a commit that referenced this pull request May 16, 2015
Fix dynamic-html positioning of parameters
@fehguy fehguy merged commit 919bab0 into swagger-api:develop_2.0 May 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants