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.

Sign up for GitHub

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

Jump to bottom

HTML entities such us   or © are not supported #6

Closed
barsch opened this issue Jan 19, 2021 · 1 comment · Fixed by #21
Closed

HTML entities such us   or © are not supported #6

barsch opened this issue Jan 19, 2021 · 1 comment · Fixed by #21

Comments

@barsch
Copy link
Contributor

barsch commented Jan 19, 2021

currently fails with:

lxml.etree.XMLSyntaxError: Entity 'copy' not defined, line 853, column 19

see also:
https://stackoverflow.com/questions/31210655/nbsp-text-not-handled-by-the-lxml
@FelixSchwarz
Copy link
Owner

Yup, should be handled but isn't. Contributions welcome :-)

@barsch barsch mentioned this issue Jan 20, 2021
Closed
@FelixSchwarz FelixSchwarz mentioned this issue Mar 11, 2021
Closed
@caseyjhol caseyjhol mentioned this issue Aug 17, 2022
Merged
caseyjhol added a commit to caseyjhol/mjml-python that referenced this issue Aug 19, 2022
@caseyjhol
Use BeautifulSoup instead of lxml for MJML/HTML parsing.
60754ec 
* Fixes FelixSchwarz#6
* Fixes FelixSchwarz#10
* Fixes FelixSchwarz#14
* Fixes FelixSchwarz#19
FelixSchwarz pushed a commit that referenced this issue Aug 22, 2022
@caseyjhol @FelixSchwarz
Use BeautifulSoup instead of lxml for MJML/HTML parsing.
54bf3b7 
* Fixes #6
* Fixes #10
* Fixes #14
* Fixes #19
@sh-at-cs sh-at-cs mentioned this issue Feb 19, 2024
Closed
@FelixSchwarz FelixSchwarz mentioned this issue Jul 9, 2024
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use BeautifulSoup instead of lxml for MJML/HTML parsing caseyjhol/mjml-python
2 participants
@FelixSchwarz @barsch