Skip to content

docs(part-6): modernize router v6 syntax & nesting#4842

Closed
BonaventureCJ wants to merge 1 commit into
reduxjs:masterfrom
BonaventureCJ:docs/v6-route-cleanup-and-nesting
Closed

docs(part-6): modernize router v6 syntax & nesting#4842
BonaventureCJ wants to merge 1 commit into
reduxjs:masterfrom
BonaventureCJ:docs/v6-route-cleanup-and-nesting

Conversation

@BonaventureCJ
Copy link
Copy Markdown
Contributor

Cleanup:

Remove the deprecated component to fix compatibility issues with React Router v6.

Improve Readability:

Refactored the flat route list into a nested structure to enhance readability and ensure consistency with the official tutorial CodeSandbox.

These updates prevent runtime errors for learners using current library versions and demonstrate the preferred pattern for descendant routes.


name: 📝 Documentation Fix
about: Fixing a problem in an existing docs page

Checklist

  • Is there an existing issue for this PR?
    • link issue here
  • Have the files been linted and formatted?

What docs page needs to be fixed?

  • Section:
  • Page:

What is the problem?

What changes does this PR make to fix the problem?

Remove the deprecated <Redirect> component to fix compatibility issues
with React Router v6. Refactored the flat route list into a nested
<Routes> structure to enhance readability and ensure consistency with
the official tutorial CodeSandbox.

These updates prevent runtime errors for learners using current library
versions and demonstrate the preferred pattern for descendant routes.
@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci Bot commented Feb 3, 2026

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 3, 2026

Deploy Preview for redux-docs ready!

Name Link
🔨 Latest commit 5778569
🔍 Latest deploy log https://app.netlify.com/projects/redux-docs/deploys/698222108c700e00076c2838
😎 Deploy Preview https://deploy-preview-4842--redux-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@timdorr
Copy link
Copy Markdown
Member

timdorr commented Feb 3, 2026

Actually, that is fine as-is. It's just a snippet of code out of the larger file, so it doesn't need to be wrapped.

@timdorr timdorr closed this Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants