Skip to content

Conversation

@TerryHowe
Copy link
Contributor

@TerryHowe TerryHowe commented Nov 20, 2025

@TerryHowe TerryHowe mentioned this pull request Nov 26, 2025
Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

👍

@scottrigby scottrigby added the bug label Nov 27, 2025
@scottrigby
Copy link
Member

scottrigby commented Nov 27, 2025

Question, did the blog rss feed previously work for languages? For example, this works on the current site https://deploy-preview-1948--helm-merge.netlify.app/ko/blog/rss.xml, but this doesn't https://deploy-preview-1948--helm-merge.netlify.app/ko/blog/index.xml. I'm not sure if it ever did though.

@TerryHowe
Copy link
Contributor Author

Question, did the blog rss feed previously work for languages? For example, this works on the current site https://deploy-preview-1948--helm-merge.netlify.app/ko/blog/rss.xml, but this doesn't https://deploy-preview-1948--helm-merge.netlify.app/ko/blog/index.xml. I'm not sure if it ever did though.

No, the previous site did not i18n the blog. Personally, I don't think we should i18n the blog just because of value/complexity.

@TerryHowe TerryHowe merged commit 86c1522 into helm:main Nov 27, 2025
3 checks passed
@TerryHowe TerryHowe deleted the fix-redirect-old-rss-feed-to-new branch November 27, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RSS feed broken

2 participants