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

Support for module:content #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

deian
Copy link

@deian deian commented Mar 23, 2013

I added support for content:encoded that is pretty widely used to integrate HTML content in RSS feeds.

I'm currently using it in LearnByHacking. See for example https://www.learnbyhacking.org/posts
and the corresponding code: https://github.com/scslab/lbh

I can add support for ther other stuff mentioned in the module (e.g., content:item), but I don't think they're as widely used as content encoded.

[Sorry for bumping the version, I can drop it if you wish]

@basvandijk
Copy link
Collaborator

Thanks for the patch but I'm not sure I should merge this since in 2006 Bjorn removed (c39dffc) the ContentEncoded constructor from rss with the explanation that the descriptor tag is the recommended way to include HTML content. Note that I'm not an RSS expert. Has this recommendation changed over the years?

@deian
Copy link
Author

deian commented Mar 26, 2013

I am not an RSS expert either, but judgin from
https://developer.mozilla.org/en-US/docs/RSS/Article/Why_RSS_Content_Module_is_Popular_-_Including_HTML_Contents
and inspecting RSS feeds on Wordpress (https://donsbot.wordpress.com/feed/) it seems like people are still using it.
Though including it in description as CDATA is common too, so I understand if you don't merge this.

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