Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: update incorrect link address #70836

Merged
merged 2 commits into from
Oct 11, 2024
Merged

docs: update incorrect link address #70836

merged 2 commits into from
Oct 11, 2024

Conversation

lonr
Copy link
Contributor

@lonr lonr commented Oct 6, 2024

What?

Updated the link in the markdown file to point to the correct location.

Why?

> **Good to know**: Dynamic Segments are equivalent to [Dynamic Routes](/docs/app/building-your-application/routing/dynamic-routes) in the `pages` directory.

Dynamic Routes in the pages directory.

However, the link thereof points to the markdown file of the app router.

How?

Changed the link from

/docs/app/building-your-application/routing/dynamic-routes
to
/docs/pages/building-your-application/routing/dynamic-routes.

@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Oct 6, 2024
@ijjk
Copy link
Member

ijjk commented Oct 6, 2024

Allow CI Workflow Run

  • approve CI run for commit: cb30c1a

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

@leerob leerob changed the title Correct link address docs: update incorrect link address Oct 11, 2024
@leerob leerob merged commit bf666b2 into vercel:canary Oct 11, 2024
39 checks passed
@lonr lonr deleted the patch-2 branch October 11, 2024 02:19
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants