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

docs: add explicit createError import - #6050

Merged
pi0 merged 2 commits into
nuxt:mainfrom
mtskf:patch-1
Jul 22, 2022
Merged

docs: add explicit createError import#6050
pi0 merged 2 commits into
nuxt:mainfrom
mtskf:patch-1

Conversation

@mtskf

Copy link
Copy Markdown
Contributor

❓ Type of change

  • [x ] 📖 Documentation (updates to the documentation or readme)

📚 Description

createError should be imported from h3 otherwise Nuxt will throw an error:

[nuxt] [request error] createError is not defined

📝 Checklist

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

`createError` should be imported from `h3`
@netlify

netlifyBot commented Jul 21, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs ready!

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

@danielroe

Copy link
Copy Markdown
Member

What version of Nuxt are you on?

@mtskf

Copy link
Copy Markdown
ContributorAuthor

@danielroe 3.0.0-rc.6

@danielroe

Copy link
Copy Markdown
Member

Try edge releases. It will be auto imported in the next release.

@pi0pi0 changed the title Update create-error.mddocs: add explicit createError importJul 22, 2022
@pi0pi0 reopened this Jul 22, 2022
@pi0

pi0 commented Jul 22, 2022

Copy link
Copy Markdown
Member

Thanks for the PR @mtskf. As @danielroe pointed out, the auto import feature for createError is merged with #4539 but still in the edge channel. I've moved your solution to a temporary banner until next release.

@pi0
pi0 merged commit 38158cf into nuxt:mainJul 22, 2022
@pi0pi0 mentioned this pull request Aug 5, 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

@mtskf@danielroe@pi0