Spice up site_template feed #2438
Conversation
| @@ -15,6 +18,12 @@ layout: none | |||
| <pubDate>{{ post.date | date: "%a, %d %b %Y %H:%M:%S %z" }}</pubDate> | |||
| <link>{{ post.url | prepend: site.baseurl | prepend: site.url }}</link> | |||
| <guid isPermaLink="true">{{ post.url | prepend: site.baseurl | prepend: site.url }}</guid> | |||
| {% for tag in post.tags %} | |||
| <category>{{ tag | xml_escape }}</category> | |||
There was a problem hiding this comment.
If tags aren't categories, why include them here?
There was a problem hiding this comment.
Honestly, the reason it is included here is because that's how it is in the jekyllrb.com feed template. If it makes more sense to leave this out that'd be an easy fix, but is there a reason it is included there and not here?
There was a problem hiding this comment.
@benbalter, have any thoughts here? I'm inclined to accept even if the semantics are kind of off.
There was a problem hiding this comment.
I'm inclined to accept even if the semantics are kind of off.
If that's the only hold up, I can just leave out tags.
The only reason I did this is because that's the way jekyllrb.com is handling tags.
|
I'm totally up for this. @benbalter, any thoughts? |
|
👍 |
Added some features to the template feed that are used by jekyllrb.com