Skip to content
New issue

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

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

Already on GitHub? # to your account

[dynamicIO] track Prerender environment name during dev #71165

Merged
merged 1 commit into from
Oct 12, 2024

Conversation

gnoff
Copy link
Contributor

@gnoff gnoff commented Oct 11, 2024

When rendering in dev we track environment name to pass along server logs to the browser. Today the environment name is the default "Server" however when dynamicIO is enabled we want to have logs in the first Task where rendering starts be tagged with the environment "Prerender". This will help us understand what work is happening in the "Prerender" phase and what is happening in the dynamic "Server" phase while developing in dev mode.v

@ijjk ijjk added created-by: Next.js team PRs by the Next.js team. type: next labels Oct 11, 2024
When rendering in dev we track environment name to pass along server logs to the browser. Today the environment name is the default "Server" however when `dynamicIO` is enabled we want to have logs in the first Task where rendering starts be tagged with the environment "Prerender". This will help us understand what work is happening in the "Prerender" phase and what is happening in the dynamic "Server" phase while developing in dev mode.v
@gnoff gnoff force-pushed the dynamic-io-dev-environment branch from 4865619 to 3357685 Compare October 11, 2024 21:58
@ijjk
Copy link
Member

ijjk commented Oct 11, 2024

Tests Passed

@gnoff gnoff requested a review from sebmarkbage October 11, 2024 22:49
@gnoff gnoff merged commit 2f195e3 into vercel:canary Oct 12, 2024
84 of 85 checks passed
@gnoff gnoff deleted the dynamic-io-dev-environment branch October 12, 2024 05:00
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 26, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants