Skip to content

Conversation

liketiger
Copy link
Contributor

Documentation Update

This PR improves the dynamic routes example in the project-structure.mdx page.

Changes

  • Updated catch-all segments example:
    • Before: /shop/clothing/shoes
    • After: /shop/clothing
  • Adjusted follow-up examples for consistency:
    • /shop/a, /shop/a/b/shop/clothing, /shop/clothing/shirts

Why

  • Provides a clearer and more intuitive demonstration of how catch-all segments work.
  • Keeps the example consistent with other dynamic routes documentation.

Updated dynamic routes example in `project-structure.mdx`. (catch-all segments)

Changed example from `/shop/clothing/shoes` to `/shop/clothing` for clarity
and consistency. This makes the catch-all segment usage more intuitive:
`/shop/a`, `/shop/a/b` → `/shop/clothing`, `/shop/clothing/shirts`.
@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Sep 22, 2025
@ijjk
Copy link
Member

ijjk commented Sep 22, 2025

Allow CI Workflow Run

  • approve CI run for commit: 7818cd0

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@icyJoseph icyJoseph merged commit 8377773 into vercel:canary Sep 23, 2025
65 checks passed
icyJoseph added a commit that referenced this pull request Sep 29, 2025
## Documentation Update

This PR improves the dynamic routes example in the
`project-structure.mdx` page.

### Changes
- Updated catch-all segments example:
  - Before: `/shop/clothing/shoes`
  - After:  `/shop/clothing`
- Adjusted follow-up examples for consistency:
  - `/shop/a`, `/shop/a/b` → `/shop/clothing`, `/shop/clothing/shirts`

### Why
- Provides a clearer and more intuitive demonstration of how catch-all
segments work.
- Keeps the example consistent with other dynamic routes documentation.

Co-authored-by: Joseph <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Related to Next.js' official documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants