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

Fix feeds #215

Closed
wants to merge 6 commits into from
Closed

Fix feeds #215

wants to merge 6 commits into from

Conversation

Miguelos
Copy link
Contributor

Update of RSS and Atom feeds to match standards.

Changes:

  • Updated self link to point site.JB.atom_path instead of site.atom_path.
  • Updated date formats to RFC 3339 for Atom and RFC 822 for RSS.

Sources:

http://cyber.law.harvard.edu/rss/rss.html
http://pixelcog.com/blog/2013/jekyll-from-scratch-core-architecture
http://validator.w3.org
http://www.w3.org/2005/Atom

@fujii
Copy link

fujii commented May 25, 2014

Same issue.
#157

id must end with /
@groundh0g groundh0g added this to the v 0.5.0 milestone Apr 5, 2015
@groundh0g groundh0g self-assigned this Apr 5, 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.

3 participants