Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix waiting for server prefetch in server renderer plus async refactor #11638

Conversation

deleteme
Copy link
Contributor

Refactors renderComponentVNode to be async, which allows the implementation to be a little more simple and less repetitive.

❗ This PR is intended to be merged into #10893, not the main brach, in order to amend it to satisfy feedback.

… option w/async setup

This fixes a bug where the serverPrefetch option would not be called
with an async setup method.

A `prefetches` reference was created while `instance.sp` was `null`.
However, with an async setup, `instance.sp` has the serverPrefetch
methods only after setup resolves.

The fix is achieved by waiting to for setup to resolve before trying to
access the the serverPrefetch options from `instance.sp`.
Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for vue-next-template-explorer failed.

Name Link
🔨 Latest commit 4ec094b
🔍 Latest deploy log https://app.netlify.com/sites/vue-next-template-explorer/deploys/66bf7e161ccc5d000856b6cd

Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for vue-sfc-playground failed.

Name Link
🔨 Latest commit 4ec094b
🔍 Latest deploy log https://app.netlify.com/sites/vue-sfc-playground/deploys/66bf7e1664889d00086f017e

@deleteme deleteme closed this Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant