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

fix(nuxt)!: only add $f fetch prefix to auto-keys - #8852

Merged
pi0 merged 3 commits into
mainfrom
fix/fetch-prefix
Nov 10, 2022
Merged

fix(nuxt)!: only add $f fetch prefix to auto-keys#8852
pi0 merged 3 commits into
mainfrom
fix/fetch-prefix

Conversation

@danielroe

@danielroedanielroe commented Nov 9, 2022

Copy link
Copy Markdown
Member

🔗 Linked issue

resolvesnuxt/nuxt#15337

❓ 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

It's tricky to use refreshNuxtData to refresh a useFetch with an explicit key as there is currently an undocumented $f prefix added. I think we can choose only to add this prefix with an auto-key, meaning that users can manually invalidate the key more easily.

📝 Checklist

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

@danielroedanielroe added the bug Something isn't working label Nov 9, 2022
@danielroe
danielroe requested a review from pi0November 9, 2022 13:04
@danielroedanielroe self-assigned this Nov 9, 2022
@codesandbox

Copy link
Copy Markdown

CodeSandbox logoCodeSandbox logo Open in CodeSandbox Web Editor | VS Code | VS Code Insiders

@netlify

netlifyBot commented Nov 9, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs canceled.

NameLink
🔨 Latest commit535da43
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/636bab6278583c000886c65f

pi0
pi0 approved these changes Nov 10, 2022
@pi0
pi0 merged commit 1709934 into mainNov 10, 2022
@pi0
pi0 deleted the fix/fetch-prefix branch November 10, 2022 08:39
@pi0pi0 mentioned this pull request Nov 15, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

3.xbugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

clearNuxtData does not work with useFetch and a provided key

2 participants

@danielroe@pi0