Skip to content

Comments

Make generated RSS valid#862

Merged
jtojnar merged 1 commit intomasterfrom
fix-rss
Feb 5, 2017
Merged

Make generated RSS valid#862
jtojnar merged 1 commit intomasterfrom
fix-rss

Conversation

@jtojnar
Copy link
Member

@jtojnar jtojnar commented Feb 5, 2017

To make it pass validation, the following was needed:

  1. Adding channel description
  2. Removing HTML tags from title
  3. Setting GUID
  4. Correctly setting lastPubDate

Additionally, self URL was added as suggested by the validator.

No problems with encoding detected after adding feed suggested in original issue.

Closes: #492

To make it pass validation, the following was needed:

1. Adding channel description
2. Removing HTML tags from title
3. Setting GUID
4. Correctly setting `lastPubDate`

Additionally, `self` URL was added as suggested by the [validator].

No problems with encoding detected after adding [feed suggested] in original issue.

Closes: #492

[validator]: https://validator.w3.org/feed/
[feed suggested]: https://alphaomegaalpha.wordpress.com/feed/
@jtojnar jtojnar merged commit aa1feb0 into master Feb 5, 2017
@jtojnar jtojnar deleted the fix-rss branch February 5, 2017 11:25
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.

1 participant