Skip to content

[Java] Use triple mustache for vendorExtensions.extraAnnotation#3825

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
wienczny:tripleMustacheExtraAnnotation
Sep 20, 2016
Merged

[Java] Use triple mustache for vendorExtensions.extraAnnotation#3825
wing328 merged 1 commit intoswagger-api:masterfrom
wienczny:tripleMustacheExtraAnnotation

Conversation

@wienczny
Copy link
Copy Markdown
Contributor

Triple mustache is required because annotations may contain chars like "=" that
would be mistakenly encoded. See #3754 for an example

Triple mustache is required because annotations may contain chars like "=" that
would be mistakenly encoded.
@wing328 wing328 added this to the v2.2.2 milestone Sep 19, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Sep 20, 2016

@wienczny thanks for the PR, which has been merged into master.

@wing328 wing328 merged commit 6ac5a2c into swagger-api:master Sep 20, 2016
@wienczny wienczny deleted the tripleMustacheExtraAnnotation branch September 20, 2016 13:20
wing328 pushed a commit that referenced this pull request Jan 13, 2017
…ion. (#4553)

This covers the usages that weren't fixed in #3825

Triple mustache is required because annotations may contain chars like
"=" that would be mistakenly encoded.
@wing328 wing328 changed the title Use triple mustache for vendorExtensions.extraAnnotation [Java] Use triple mustache for vendorExtensions.extraAnnotation Feb 20, 2017
davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
…ion. (swagger-api#4553)

This covers the usages that weren't fixed in swagger-api#3825

Triple mustache is required because annotations may contain chars like
"=" that would be mistakenly encoded.
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.

2 participants