Skip to content

add tags with API name to java @Api annotation#3189

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
thebignet:tagName
Jun 27, 2016
Merged

add tags with API name to java @Api annotation#3189
wing328 merged 1 commit intoswagger-api:masterfrom
thebignet:tagName

Conversation

@thebignet
Copy link
Contributor

No description provided.

@wing328
Copy link
Contributor

wing328 commented Jun 21, 2016

@thebignet thanks for the PR on spring mvc and spring boot generator. I've just merged #3133 into master.

Please pull the latest master and rebase the change accordingly.

@cbornet
Copy link
Contributor

cbornet commented Jun 21, 2016

I think it would be better to tag the operations with tags={ {{#vendorExtensions.x-tags}}"{{tag}}",{{/vendorExtensions.x-tags}} like what is done in jax-rs jersey template

@thebignet
Copy link
Contributor Author

Tried it out on swagger-ui and got even better results

@cbornet
Copy link
Contributor

cbornet commented Jun 25, 2016

👍

@wing328 wing328 added this to the v2.2.0 milestone Jun 27, 2016
@wing328 wing328 merged commit 9ec61ba into swagger-api:master Jun 27, 2016
@wing328
Copy link
Contributor

wing328 commented Jun 27, 2016

@thebignet thanks for the PR. Next time, please also update the Petstore sample.

@thebignet thebignet deleted the tagName branch July 1, 2016 20:30
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.

3 participants