Skip to content

Conversation

@bryanbraun
Copy link
Contributor

Hey @KyleAMathews. πŸ‘‹

This PR fixes a few things in your RSS feed, to make it more accurate and findable:

  • Updates the feed title: Your Site's RSS Feed β†’ Bricolage
  • Removes the example external feed (https://feeds.feedburner.com/gatsby/blog), which doesn't point to your site. I think it was accidentally copied over from the gatsby-plugin-feed docs previously.
  • Removes the match criteria
    • This is because the <link type="application/rss+xml" ...> isn't updated on client-side transitions. As a result, if you enter the site on the home-page, and navigate to /blog, you won't see the <link> on the /blog page, which makes the feed less discoverable. I think the easiest solution is to include the <link> on all pages by default (as recommended here), and removing the match property does that.

@netlify
Copy link

netlify bot commented Nov 3, 2023

βœ… Deploy Preview for monumental-begonia-4d26ee ready!

Name Link
πŸ”¨ Latest commit 9f3caf9
πŸ” Latest deploy log https://app.netlify.com/sites/monumental-begonia-4d26ee/deploys/654505ee5bcb3b00087b5cef
😎 Deploy Preview https://deploy-preview-83--monumental-begonia-4d26ee.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@KyleAMathews
Copy link
Owner

Thanks!

@KyleAMathews KyleAMathews merged commit 5631b00 into KyleAMathews:master Nov 3, 2023
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.

2 participants