Skip to content
This repository has been archived by the owner on Nov 30, 2024. It is now read-only.

Update rss.xml and atom.xml #157

Closed
wants to merge 2 commits into from
Closed

Conversation

dieyushi
Copy link

@dieyushi dieyushi commented May 7, 2013

update those two file to match RSS 2.0 Specification and RFC 4287.


{% for post in site.posts %}
{% for post in site.posts limit:15 %}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's better to give an option in _config.yml, like named rss_limit.

@dieyushi
Copy link
Author

dieyushi commented Jun 4, 2013

@Lax yeah, it's better.

@fujii fujii mentioned this pull request May 25, 2014
<item>
<title>{{ post.title }}</title>
<description>{{ post.content | xml_escape }}</description>
<link>{{ site.production_url }}{{ post.url }}</link>
<guid>{{ site.production_url }}{{ post.id }}</guid>
<pubDate>{{ post.date | date_to_xmlschema }}</pubDate>
<pubDate>{{ post.date | date: "%a, %d %b %Y %H:%M:%S %z" }}</pubDate>
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm using the liquid template date _to_rfc822 (http://jekyllrb.com/docs/templates/)[http://jekyllrb.com/docs/templates/]:

               <pubDate>{{ post.date | date_to_rfc822 }}</pubDate>

Source in #215

@groundh0g groundh0g added this to the v 0.5.0 milestone Apr 4, 2015
@groundh0g groundh0g self-assigned this Apr 4, 2015
@groundh0g
Copy link
Collaborator

Unfortunately, it's been too long to merge this PR as-is. I've made the edits from #157 and #215 locally and am testing tonight. Hope to commit and close soon. Thanks for your contribution. Sorry for the delay in accepting it.

@groundh0g
Copy link
Collaborator

Testing is complete. Works as expected. Thanks for the patch! Will merge the changes to master soonish.

groundh0g added a commit to groundh0g/jekyll-bootstrap that referenced this pull request May 23, 2015
. In a nutshell: limit RSS feeds to 20, correct some bathing issues, format timestamps, shorten TTL.
@groundh0g
Copy link
Collaborator

Incorporated into #272.

@groundh0g groundh0g closed this May 23, 2015
groundh0g added a commit that referenced this pull request May 24, 2015
groundh0g added a commit to groundh0g/jekyll-bootstrap that referenced this pull request May 25, 2015
aterai pushed a commit to aterai/jekyll-bootstrap that referenced this pull request Mar 11, 2019
. In a nutshell: limit RSS feeds to 20, correct some bathing issues, format timestamps, shorten TTL.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants