Skip to content

Conversation

@babblebey
Copy link

@babblebey babblebey commented Jan 20, 2026

This pull request updates the routing documentation for React to clarify the correct file structure for dynamic route segments. The example now uses a slash (/) instead of a dot (.) to indicate dynamic parameters, aligning with the expected convention.

Change Made

  • Corrected the example route filename from posts.$postId.tsx to posts/$postId.tsx in docs/start/framework/react/guide/routing.md to accurately reflect dynamic route segment syntax.

Summary by CodeRabbit

  • Documentation
    • Updated the nested routing guide with the correct dynamic route file naming convention in examples.

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions github-actions bot added the documentation Everything documentation related label Jan 20, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

Updates the example in the Nested Routing section from a dynamic route file named posts.$postId.tsx to posts/$postId.tsx, switching from dot notation to a slash-based path notation in the documented routing example.

Changes

Cohort / File(s) Summary
Documentation Update
docs/start/framework/react/guide/routing.md
Updated dynamic route file naming convention from posts.$postId.tsx to posts/$postId.tsx in the Nested Routing example

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

documentation

Suggested reviewers

  • LadyBluenotes
  • Insik-Han

Poem

🐰 A dot becomes a slash so fine,
The routing guide now will align,
With clearer paths through nested ways,
Our documentation brighter stays! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(doc): correct file path format for posts route in routing guide' accurately summarizes the main change: correcting the file path format in documentation from dot notation to slash notation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant