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/fix cookie store #71702

Merged
merged 3 commits into from
Oct 30, 2024
Merged

Docs/fix cookie store #71702

merged 3 commits into from
Oct 30, 2024

Conversation

notwatermango
Copy link
Contributor

The cookieStore was initialized from cookies()

So when calling methods use cookieStore.get instead of cookieStore().get. Applies for all other methods

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

ijjk commented Oct 23, 2024

Allow CI Workflow Run

  • approve CI run for commit: a8ce228

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

@ijjk ijjk enabled auto-merge (squash) October 30, 2024 18:51
@ijjk ijjk merged commit 90db858 into vercel:canary Oct 30, 2024
38 checks passed
stipsan pushed a commit to sanity-io/next.js that referenced this pull request Nov 6, 2024
The `cookieStore` was initialized from `cookies()`

So when calling methods use `cookieStore.get` instead of
`cookieStore().get`. Applies for all other methods

---------

Co-authored-by: JJ Kasper <jj@jjsweb.site>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 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.

2 participants