-
Notifications
You must be signed in to change notification settings - Fork 57
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
HTML to Atom: title element has encoded HTML #629
Comments
Oh I just realized it's not the first youtube |
Huh! Thanks for the nudge. This is an odd one! It doesn't reproduce locally for me at all, with either the |
Looked at this again, I'm now able to reproduce it locally. Output is still a bit different, I'm guessing that's because I'm using a different HTML parser locally vs in prod. I suspect we're generating title from HTML content, then ellipsizing, and we end up with just the opening |
Fixed! |
Minor, but I noticed with this feed that one of my notes has encoded HTML getting into the
<title>
:<title>I like this CSS image reset after watching Kevin Powell’s walkthrough. <...</title>
The original post uses
e-content
and I'm guessing Granary is using the parsedhtml
attribute for the title. A possible solution might be to usevalue
attribute instead, for posts that don't have aname
of course:The text was updated successfully, but these errors were encountered: