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

fix: correct root redirect when trailingSlash = false #879

Merged
merged 2 commits into from
Dec 6, 2021

Conversation

ascorbic
Copy link
Contributor

@ascorbic ascorbic commented Dec 3, 2021

Summary

The root level redirect when NEXT_LOCALE is set was incorrectly defined when trailingSlash was set to false.

Test plan

  1. Set a test site to trailingSlash = false
  2. Check that there is no redirect created with from = ''

Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal

Fixes #872

Standard checks:

  • Check the Deploy Preview's Demo site for your PR's functionality
  • Add docs when necessary

🧪 Once merged, make sure to update the version if needed and that it was published correctly.

@netlify
Copy link

netlify bot commented Dec 3, 2021

✔️ Deploy Preview for netlify-plugin-nextjs-static-root-demo ready!

🔨 Explore the source changes: a81bc68

🔍 Inspect the deploy log: https://app.netlify.com/sites/netlify-plugin-nextjs-static-root-demo/deploys/61ade362f872c2000757b9d6

😎 Browse the preview: https://deploy-preview-879--netlify-plugin-nextjs-static-root-demo.netlify.app

@netlify
Copy link

netlify bot commented Dec 3, 2021

✔️ Deploy Preview for netlify-plugin-nextjs-nx-monorepo-demo ready!

🔨 Explore the source changes: a81bc68

🔍 Inspect the deploy log: https://app.netlify.com/sites/netlify-plugin-nextjs-nx-monorepo-demo/deploys/61ade3625cee1a0008305f2f

😎 Browse the preview: https://deploy-preview-879--netlify-plugin-nextjs-nx-monorepo-demo.netlify.app/

@github-actions github-actions bot added the type: bug code to address defects in shipped code label Dec 3, 2021
@kodiakhq kodiakhq bot merged commit 3c6b10b into main Dec 6, 2021
@kodiakhq kodiakhq bot deleted the mk/redirect-i18n-slash branch December 6, 2021 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge type: bug code to address defects in shipped code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Invalid redirect rule generated for i18n cookie
2 participants