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

docs: add note about fetching data on initial load - #7120

Merged
pi0 merged 3 commits into
mainfrom
docs/client-data-fetching-note
Sep 3, 2022
Merged

docs: add note about fetching data on initial load#7120
pi0 merged 3 commits into
mainfrom
docs/client-data-fetching-note

Conversation

@danielroe

Copy link
Copy Markdown
Member

🔗 Linked issue

resolvesnuxt/nuxt#14694

❓ 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

This adds a note about a limitation of server: false data fetching and updates a couple of other minor inaccuracies..

📝 Checklist

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

@danielroedanielroe added the documentation Improvements or additions to documentation label Aug 31, 2022
@danielroe
danielroe requested a review from pi0August 31, 2022 20:41
@danielroedanielroe self-assigned this Aug 31, 2022
@netlify

netlifyBot commented Aug 31, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs ready!

NameLink
🔨 Latest commitdfc5db4
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/631334127de7ca000900c897
😎 Deploy Previewhttps://deploy-preview-7120--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.

pi0
pi0 approved these changes Sep 3, 2022

@pi0pi0 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing this out. It seems a buggy behavior to me since we could make data available after awaiting for client-side only. But let's document warning until fixing (read topic. hydration is tricky indeed)

@pi0
pi0 merged commit 0f1a746 into mainSep 3, 2022
@pi0
pi0 deleted the docs/client-data-fetching-note branch September 3, 2022 11:02
@danielroe

Copy link
Copy Markdown
MemberAuthor

I don't think we can wait in setup, or we will cause a hydration error.

This was referenced Sep 9, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

3.xdocumentationImprovements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

useAsyncData's data._rawValue is null

3 participants

@danielroe@pi0@DamianGlowala