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 i18n for apps with a middleware #61561

Open
wants to merge 3 commits into
base: canary
Choose a base branch
from

Conversation

cornedor
Copy link

@cornedor cornedor commented Feb 2, 2024

Fixes #58583

Every pass of the handleRoute function modifies the parsedUrl object. This caused the /next/_data url to be modified too early, and be stripped of any locale data.

Added a test that would fail before.

@krzysiek1507
Copy link

Hi! Any update on this one?

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.

I18n breaks when using a middleware
4 participants