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

Update proxy host when proxy base path exists as well #2299

Merged
merged 4 commits into from
Mar 16, 2017
Merged

Update proxy host when proxy base path exists as well #2299

merged 4 commits into from
Mar 16, 2017

Conversation

marla-singer
Copy link
Contributor

@marla-singer marla-singer commented Mar 15, 2017 •
edited

Closes #2286
Closes #2298

  • Update Swagger host only if API has proxyBase path
  • submit_field was replaced from API doc. Get value of submit_methodsfrom apiDocs

@marla-singer marla-singer added this to the Sprint 38 milestone Mar 16, 2017
@marla-singer
Copy link
Contributor Author

@apinf/developers Ready for review

@brylie
Copy link
Contributor

brylie commented Mar 16, 2017 •
edited

In addition to the issue links, please also describe your changes in the PR descriptions. That makes it a bit easier for the reviewer to understand what they are looking at.

@brylie brylie self-assigned this Mar 16, 2017
@brylie brylie merged commit ffcb517 into develop Mar 16, 2017
@brylie brylie deleted the bugfix/try-out-button branch March 16, 2017 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@brylie brylie

Labels
None yet
Projects
None yet
Milestone
Sprint 38
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@marla-singer @brylie