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.

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

Bugfix: <3 is not a valid html tag start, it's an emoji. #171

Merged
merged 2 commits into from
Apr 12, 2016
Merged

Bugfix: <3 is not a valid html tag start, it's an emoji. #171

merged 2 commits into from
Apr 12, 2016

Conversation

vvoznesensky
Copy link
Contributor

Browsers treat <3 as text, not as a start of html tag.

@etrepum
Copy link
Member

etrepum commented Apr 11, 2016

Do you have time to add a test to this PR as well?

@vvoznesensky
Copy link
Contributor Author

I have time but I have no idea where to put this into mochiweb_html_tests.erl .

@etrepum
Copy link
Member

etrepum commented Apr 11, 2016

Just add a new test

@vvoznesensky
Copy link
Contributor Author

Have the test, please.

@etrepum
Copy link
Member

etrepum commented Apr 12, 2016

Looks great, nice work

@etrepum etrepum merged commit dd26e09 into mochi:master Apr 12, 2016
@vvoznesensky
Copy link
Contributor Author

BTW, I'm looking for a great nice work.

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