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

docs: fix typo in defineEventHandler example - #6741

Merged
pi0 merged 1 commit into
nuxt:mainfrom
zhangzhibang0309:patch-1
Aug 18, 2022
Merged

docs: fix typo in defineEventHandler example#6741
pi0 merged 1 commit into
nuxt:mainfrom
zhangzhibang0309:patch-1

Conversation

@zhangzhibang0309

@zhangzhibang0309zhangzhibang0309 commented Aug 18, 2022•
edited
Loading

Copy link
Copy Markdown
Contributor
export default anync( should delete ) defineEventHandler(async (event) => {
const data = await useStorage().getItem('redis:test')
return data
})

This function should not be called with async。

🔗 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

📝 Checklist

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

@zhangzhibang0309
Update 9.server-routes.md
23913f7
```
export default defineEventHandler(async (event) => {
const data = await useStorage().getItem('redis:test')
return data
})
```
This function should not be called with async。
@netlify

netlifyBot commented Aug 18, 2022•
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for nuxt3-docs ready!

NameLink
🔨 Latest commit23913f7
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/62fe09b048ea5900084e3fe1
😎 Deploy Previewhttps://deploy-preview-6741--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.

@zhangzhibang0309zhangzhibang0309 changed the title Update 9.server-routes.mdDocs: Remove async from function callsAug 18, 2022
@danielroedanielroe changed the title Docs: Remove async from function callsdocs: fix typo in defineEventHandler exampleAug 18, 2022
@danielroedanielroe added the documentation Improvements or additions to documentation label Aug 18, 2022
@pi0
pi0 merged commit 40f0c82 into nuxt:mainAug 18, 2022
@pi0

pi0 commented Aug 18, 2022

Copy link
Copy Markdown
Member

Thanks!

@pi0pi0 mentioned this pull request Aug 26, 2022
Merged
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@danielroedanielroeAwaiting requested review from danielroe

Assignees

No one assigned

Labels

3.xdocumentationImprovements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zhangzhibang0309@pi0@danielroe