-
-
Notifications
You must be signed in to change notification settings - Fork 4
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RSS feed to the blog #16
Conversation
…age, include RSS icon and link in the footer
✅ Deploy Preview for spyder-svelte-website-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a subject matter expert with the mechanics of the site and upon inspection don't see anything obviously amiss here so I'll defer to your expertise on that @conradolandia . As for the functionality, I don't really use RSS myself nor does my browser support it anymore, but maybe @jondo can do a quick double-check that it works for their use case?
In the deploy preview this now works fine, thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that case, LGTM! Thanks @conradolandia and @jondo !
This PR improves the following:
/blog/feed.xml
<link>
element in the header of the blog pageFixes #12