Skip to content
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

docs(cookies): fix link to Dynamic Function #68780

Merged
merged 1 commit into from
Aug 23, 2024
Merged

docs(cookies): fix link to Dynamic Function #68780

merged 1 commit into from
Aug 23, 2024

Conversation

Juneezee
Copy link
Contributor

@Juneezee Juneezee commented Aug 12, 2024

image

There is an invalid link on the cookies page (https://nextjs.org/docs/app/api-reference/functions/cookies). Currently it links to the incorrect section. The correct link should be https://nextjs.org/docs/app/building-your-application/rendering/server-components#dynamic-functions

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Aug 12, 2024
@ijjk
Copy link
Member

ijjk commented Aug 12, 2024

Allow CI Workflow Run

  • approve CI run for commit: 8cc2712

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

1 similar comment
@ijjk
Copy link
Member

ijjk commented Aug 12, 2024

Allow CI Workflow Run

  • approve CI run for commit: 8cc2712

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

@eps1lon eps1lon requested a review from delbaoliveira August 12, 2024 08:31
@Juneezee
Copy link
Contributor Author

friendly ping @delbaoliveira

cc @leerob

@leerob leerob merged commit 99e5427 into vercel:canary Aug 23, 2024
39 checks passed
@github-actions github-actions bot added the locked label Sep 8, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants