Skip to content
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

feat: first naive implementation of an RSS feed #306

Merged
merged 4 commits into from
Jan 23, 2024
Merged

Conversation

nunopolonia
Copy link
Contributor

@nunopolonia nunopolonia commented Dec 27, 2023

Fixes SIGN-324

First naive implementation.
I know Storyblok already handles the data caching on their side, and I added some random values to the Cache-Control headers, but I probably can change them to no-cache and let them be controlled by storyblok's CDN cache.

If anyone has more knowledge / opinions on how to deal with this, let me know

Copy link

linear bot commented Dec 27, 2023

tofran
tofran previously approved these changes Jan 8, 2024
Copy link
Member

@tofran tofran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, this is cool

Just one thing, it would make sense if the description, title, etc all came from the same place.

@nunopolonia
Copy link
Contributor Author

@tofran changed the PR to include the improvements you suggested. We're now getting the channel info from the blog storyblok SEO configuration. I've also removed the condition that was allowing draft posts to be shown in the rss feed if a cookie was set

@nunopolonia nunopolonia merged commit e14cd4b into main Jan 23, 2024
2 checks passed
@nunopolonia nunopolonia deleted the sign-324-rss-feed branch January 23, 2024 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants