Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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/

@ascorbic ascorbic requested a review from tiffafoo December 3, 2021 16:27
@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