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

always link to /whats-new/feed.xml #17222

Merged
merged 1 commit into from
May 7, 2024
Merged

Conversation

sunnyzanchi
Copy link
Contributor

@sunnyzanchi sunnyzanchi commented May 7, 2024

i18n versions of this page were linking to /<locale>/whats-new/feed.xml, which doesn't exist, as the RSS feed isn't translated. this change makes sure we always properly link to the RSS feed

@sunnyzanchi sunnyzanchi requested a review from a team as a code owner May 7, 2024 19:09
@sunnyzanchi sunnyzanchi requested review from clarkmcadoo and LizBaker and removed request for a team May 7, 2024 19:09
Copy link

github-actions bot commented May 7, 2024

Hi @sunnyzanchi 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes).

@github-actions github-actions bot added the eng issues related to site functionality that requires engineering label May 7, 2024
@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues May 7, 2024
Copy link

netlify bot commented May 7, 2024

Deploy Preview for docs-website-netlify ready!

Name Link
🔨 Latest commit be46998
🔍 Latest deploy log https://app.netlify.com/sites/docs-website-netlify/deploys/663a7ce7f959e2000947a150
😎 Deploy Preview https://deploy-preview-17222--docs-website-netlify.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.

i18n versions of this page were linking to /<locale>/whats-new/feed.xml,
which doesn't exist, as the RSS feed isn't translated.
this change makes sure we always properly link to the RSS feed
@clarkmcadoo
Copy link
Contributor

still getting the 404 on this page - https://deploy-preview-17222--docs-website-netlify.netlify.app/es/whats-new/

@sunnyzanchi sunnyzanchi merged commit 75449d5 into develop May 7, 2024
17 checks passed
@sunnyzanchi sunnyzanchi deleted the sunny/fix-i18n-rss-links branch May 7, 2024 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eng issues related to site functionality that requires engineering
Projects
Docs PRs and Issues
Hero to triage
Development

Successfully merging this pull request may close these issues.

None yet

2 participants