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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid RSS data structure #49

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@snamiki1212
Copy link

snamiki1212 commented Nov 18, 2018

About RSS data structure, valid author tag position is in item tag but it seems no to be actually that馃槺
https://www.w3schools.com/xml/rss_tag_author.asp

We can generate /rss.xml from this config file, but this generated RSS is an invalid data structure so we cannot use it.

Probably best fix is to add author tag in each item tags...馃?
At least, to merge this PR and we could generate valid RSS xml.

@snamiki1212 snamiki1212 changed the title Remove author column at rss Invalid RSS data structure Nov 20, 2018

@Vagr9K Vagr9K referenced this pull request Jan 20, 2019

Merged

Winter update. #52

@Vagr9K Vagr9K closed this in #52 Jan 20, 2019

@Vagr9K

This comment has been minimized.

Copy link
Owner

Vagr9K commented Jan 20, 2019

PR #52 fixes the issue and adds the author tag to the correct position.

Validation is passing now: https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Fvagr9k.github.io%2Fgatsby-advanced-starter%2Frss.xml.

Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment