-
Notifications
You must be signed in to change notification settings - Fork 4k
Use Promise.all whenever possible #3319
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
Conversation
|
Summary of the deployments: Version 1
Version 2
Test content |
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes sense, but I don't think it'll have much impact, I think we should look more at how Next.js process components rendered as siblings that are suspending: is it serial or parallel?
Totally agree, it's just the first step. But in some cases it could still save a couple hundreds of milliseconds, that's an easy win. One suspended component will not render it's children before actually reaching the JSX component. For some nested deep in multiple I'm working on a PR that will prefetch things ASAP in the component tree (basically in the layout/page), and we then use these prefetched data instead of fetching them directly in the component. |
This should decrease TTFB for page that needs to be generated
Close RND-7316