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

fix: incorrect cookie delete argument #57526

Closed
wants to merge 10 commits into from

Conversation

asibilia
Copy link

What?

Getting the following error (from TS) when using the with-supabase example and NextJS@14

Object literal may only specify known properties, and 'name' does not exist in type 'string[]'

Screenshot 2023-10-26 at 3 06 04 PM

I was able to fix this type issue by updating:

request.cookies.delete({
  name,
  ...options,
})

to:

request.cookies.delete(name)

Fixes #57524

@asibilia asibilia requested review from a team as code owners October 26, 2023 19:07
@asibilia asibilia requested review from timeyoutakeit and StephDietz and removed request for a team October 26, 2023 19:07
@ijjk ijjk added the examples Issue was opened via the examples template. label Oct 26, 2023
@leerob
Copy link
Member

leerob commented Oct 31, 2023

#57562

@leerob leerob closed this Oct 31, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue was opened via the examples template. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect usage of request.cookies.delete in with-supabase example
3 participants