Skip to content
This repository was archived by the owner on Apr 6, 2023. It is now read-only.
/frameworkPublic archive

docs: set cookie event - #5387

Closed
ghost wants to merge 1 commit into
mainfrom
unknown repository
Closed

docs: set cookie event#5387
ghost wants to merge 1 commit into
mainfrom
unknown repository

Conversation

@ghost

@ghostghost commented Jun 10, 2022

Copy link
Copy Markdown

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

setCookie's event parameter should be req instead of res.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@netlify

netlifyBot commented Jun 10, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs ready!

NameLink
🔨 Latest commite91b088
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/62a2db2f3a081d000871d975
😎 Deploy Previewhttps://deploy-preview-5387--nuxt3-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Comment threaddocs/content/3.api/1.composables/use-cookie.md
@ghostghost closed this Jun 10, 2022
@pi0pi0 reopened this Jun 10, 2022
@pi0

pi0 commented Jun 10, 2022

Copy link
Copy Markdown
Member

Thanks for PR @ashgo-x and the tip @Diizzayy. I've reopened your PR to update example using event and dwefineEventHandler() wrapper. While it works with backward compatibility API, example is still wrong. ~> #5405

@pi0pi0 closed this Jun 10, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@pi0@Diizzayy@danielroe