search

rbren / rss-parser

A lightweight RSS parser, for Node and the browser
MIT License
1.38k stars 209 forks source link

Add `summary` field to the build-in atom parsing #192

Closed cognition9144 closed 3 years ago

cognition9144 commented 3 years ago

According to the atom spec, there's a summary field. Some existing atom feeds provide only the summary field rather than the content field. However, that doesn't violate the spec. And it's hard to urge every feed to provide a content field. If rss-parser doesn't parse summary, it might result in feeds with no real contents. I want to upstreamize it because this is not a problem of a special use case.

Related issue: https://github.com/yang991178/fluent-reader/pull/143

rbren commented 3 years ago

Awesome, thanks!

Mind just reverting dist/? I'll regenerate those as part of the release process

cognition9144 commented 3 years ago

Awesome, thanks!

Mind just reverting dist/? I'll regenerate those as part of the release process

Sure. Just in case it was out of sync.