Skip to content

Swagger routes basepath#220

Merged
ikitommi merged 3 commits intometosin:masterfrom
hoxu:swagger-routes-basepath
Feb 10, 2016
Merged

Swagger routes basepath#220
ikitommi merged 3 commits intometosin:masterfrom
hoxu:swagger-routes-basepath

Conversation

@hoxu
Copy link
Copy Markdown
Contributor

@hoxu hoxu commented Feb 10, 2016

Teach swagger-routes to heed basePath for swagger-docs path

If {:data {:basePath "/app"}} is given, default to including it as a
prefix to swagger-ui :swagger-docs path. The basePath is already passed
to swagger-docs, so this makes the swagger-ui consistent with the
generated json.

The swagger-docs path can also now be completely overridden via:
{:options {:ui {:swagger-docs "/override.json"}}}

hoxu added 3 commits February 10, 2016 11:26
If {:data {:basePath "/app"}} is given, default to including it as a
prefix to swagger-ui :swagger-docs path. The basePath is already passed
to swagger-docs, so this makes the swagger-ui consistent with the
generated json.

The swagger-docs path can also now be completely overridden via:
{:options {:ui {:swagger-docs "/override.json"}}}
ikitommi added a commit that referenced this pull request Feb 10, 2016
@ikitommi ikitommi merged commit 0b651c1 into metosin:master Feb 10, 2016
@ikitommi
Copy link
Copy Markdown
Member

Awesome, thanks!

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.

2 participants