Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[next]: ensure locale prefixed dynamic routes come before non-prefixed #13003

Merged
merged 4 commits into from
Feb 4, 2025

Conversation

ztanner
Copy link
Member

@ztanner ztanner commented Feb 3, 2025

With the experimental flag added in #12916 we added handling to add non-localized redirects for dynamic routes, because we moved the defaultLocale handling to the handle: miss case. However, we need to add these redirects after the less-specific localized routes so that those ones are matched first before the fallback case.

Copy link

changeset-bot bot commented Feb 3, 2025

🦋 Changeset detected

Latest commit: c509461

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@vercel/next Patch
vercel Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

ijjk
ijjk previously approved these changes Feb 3, 2025
@ijjk ijjk added this pull request to the merge queue Feb 4, 2025
Merged via the queue into main with commit 3875856 Feb 4, 2025
166 checks passed
@ijjk ijjk deleted the ztanner/reorder-dynamic-handling branch February 4, 2025 16:38
EndangeredMassa pushed a commit that referenced this pull request Feb 5, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## vercel@41.0.1

### Patch Changes

- [cli] remove warning `projects lists --update-required` about
deprecated node versions
([#13016](#13016))

- [cli] updat vercel/fun to 1.1.4
([#13018](#13018))

- Updated dependencies
\[[`38758561c2cfe9ebae2b1d2af506b585639a2b84`](3875856),
[`150990344d7195c72bb336614153c77d8cefb78c`](1509903)]:
    -   @vercel/next@4.5.1
    -   @vercel/node@5.1.1
    -   @vercel/remix-builder@5.2.2

## @vercel/client@15.0.1

### Patch Changes

- Updated dependencies
\[[`150990344d7195c72bb336614153c77d8cefb78c`](1509903)]:
    -   @vercel/routing-utils@5.0.3

## @vercel/frameworks@3.6.1

### Patch Changes

- Enable "demo" property for "react-router" framework
([#13008](#13008))

## @vercel/fs-detectors@5.3.7

### Patch Changes

- Updated dependencies
\[[`ed69811b310bc46347b225f516ff2f7e0817933c`](ed69811),
[`150990344d7195c72bb336614153c77d8cefb78c`](1509903)]:
    -   @vercel/frameworks@3.6.1
    -   @vercel/routing-utils@5.0.3

## @vercel/next@4.5.1

### Patch Changes

- ensure non-locale prefixed route variants come after more specific
ones ([#13003](#13003))

## @vercel/node@5.1.1

### Patch Changes

- better path-to-regexp diff logging
([#12962](#12962))

## @vercel/remix-builder@5.2.2

### Patch Changes

- better path-to-regexp diff logging
([#12962](#12962))

## @vercel/routing-utils@5.0.3

### Patch Changes

- better path-to-regexp diff logging
([#12962](#12962))

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants