Skip to content

Conversation

@ShaneK
Copy link
Member

@ShaneK ShaneK commented Dec 11, 2025

Issue number: resolves internal


What is the current behavior?

Routes with relative paths (e.g., path="tab1/*") inside root-level splat routes (path="*") do not match correctly. The parent path computation returns an incorrect value, causing routes to 404. Developers must use absolute paths (e.g., path="/tab1/*") as a workaround.

What is the new behavior?

Routes with relative paths now correctly match when the parent is a splat-only route. The computeParentPath function checks at root level for embedded wildcard routes (like tab1/*) as a fallback when no match is found at deeper levels, allowing relative paths to work correctly inside splat route parents.

Does this introduce a breaking change?

  • Yes
  • No

Other information

Current dev build:

8.7.13-dev.11765477700.112ae0a3

@ShaneK ShaneK requested a review from a team as a code owner December 11, 2025 18:41
@ShaneK ShaneK requested review from thetaPC and removed request for a team December 11, 2025 18:41
@vercel
Copy link

vercel bot commented Dec 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ionic-framework Ready Ready Preview Comment Dec 12, 2025 7:44pm

Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, minor suggestion

Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@brandyscarney brandyscarney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good fix! 🥇

@ShaneK ShaneK merged commit 60cedf0 into fix/react-router-6-paths Dec 12, 2025
49 checks passed
@ShaneK ShaneK deleted the fix/react-router-6-splat-links branch December 12, 2025 19:53
@ShaneK
Copy link
Member Author

ShaneK commented Dec 15, 2025

@ptmkenny were you able to verify if this fixes your issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: react @ionic/react package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants