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

Clean content_html and handle when it's empty #1896

Merged
merged 4 commits into from
Jul 8, 2024
Merged

Conversation

longhotsummer
Copy link
Contributor

@longhotsummer longhotsummer commented Jul 8, 2024

closes #1852
closes #1807
closes #1863
closes #1864

in all likelihood this was due to utf content; an actual HTML parse
error is rare. using parse_html_str fixes utf content issues

closes #1807
Copy link
Contributor

@actlikewill actlikewill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great

@longhotsummer longhotsummer merged commit 959ce8b into main Jul 8, 2024
9 checks passed
@longhotsummer longhotsummer deleted the content-whitespace branch July 8, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants