-
-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
[Bug]: Wrong nested path rendered in v6.0.0-beta.1 #7948
Comments
Same issue here - everything in |
The issue seems to be that a https://codesandbox.io/s/async-tdd-yvgk8?file=/src/Login.jsx (see the console) Here : react-router/packages/react-router/index.tsx Line 541 in 05c41d5
I don't think |
I think I have related issue. I have a bunch of nested routes. In /form/, I have something along the lines <Routes>
{paths.map((path) => (
<Route
key={path}
path={path}
element={<p>Step {path}</p>}
/>
))}
<Route path="*" element={<Navigate to={resolvedPaths[0]} />} />
</Routes> where
so that when you enter When updating from 6.0.0-beta.0 to 6.0.0-beta.1, this stopped working, and redirected me to |
Same issue for me... |
Fixed in #7961 |
What version of React Router are you using?
v6.0.0-beta.1
Steps to Reproduce
Codesandbox link: https://codesandbox.io/s/dreamy-cookies-1z1og
// forget-password.jsx
Expected Behavior
it should behave as v6.0.0-beta.0, the correct should be render as /auth/login.
I think its a bug on the latest build without proper checking on nested path rendering.
Actual Behavior
The nested paths are rendered with concatenating parent path with absolute path which is incorrect , the result rendered is /auth/auth/login
Using Link component or useNavigation both returning wrong nested path.
Solution now is fallback to v6.0.0-beta.0
The text was updated successfully, but these errors were encountered: