Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[csharp] Escape special characters in the API doc #4183

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

Merged
wing328 merged 2 commits into swagger-api:master from hamedrnik:escape_special_chars
Dec 16, 2016

Conversation

@hamedrnik
Copy link
Contributor

@hamedrnik hamedrnik commented Nov 14, 2016

Special characters like <, > should be HTML escaped. When the data type contains these characters (e.g. List<long?>), should be converted to the escaped version(e.g. List&lt;long?&gt;).

Special characters like <> should be HTML escaped.
Copy link
Contributor

wing328 commented Nov 15, 2016

@IceAge How did you display the document (markdown)? (< and > need to be escaped in HTML but not markdown)

Copy link
Contributor Author

@wing328 As Markdown supports inline HTML, I think open angle bracket following a close angle bracket should be escaped. Please refer to Automatic Escaping for Special Characters in Markdown documentation.

How did you display the document (markdown)?

Github

@wing328 wing328 merged commit 6af43dc into swagger-api:master Dec 16, 2016
Copy link
Contributor

wing328 commented Dec 16, 2016

(If your company is using Swagger Codegen, please consider adding your company name to this list)

hamedrnik reacted with thumbs up emoji

zxy198717 pushed a commit to zxy198717/swagger-codegen that referenced this pull request Dec 20, 2016
davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

v2.2.2

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /