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

Add onLoad gottcha note to next/script docs #33097

Merged
merged 4 commits into from
Jan 14, 2022
Merged

Conversation

callumgare
Copy link
Contributor

Add a note to the onLoad section of the next/script documentation clarifying that onLoad will not run if strategy="beforeInteractive". (That tripped me up for a bit so I thought I'd update the docs to save any one else the hassle :).)

Add a note to the onLoad section of the next/script documenation clarifying that onLoad will not run if `strategy="beforeInteractive"`.
leerob
leerob previously approved these changes Jan 8, 2022
@leerob leerob dismissed stale reviews from themself via 4c17e57 January 8, 2022 03:57
@balazsorban44
Copy link
Member

balazsorban44 commented Jan 11, 2022

Related #26343, #28889

@styfle styfle changed the title Add onLoad gottcha note to next/script doco Add onLoad gottcha note to next/script docs Jan 14, 2022
@styfle styfle changed the title Add onLoad gottcha note to next/script docs Add onLoad gottcha note to next/script docs Jan 14, 2022
@kodiakhq kodiakhq bot merged commit 5748915 into vercel:canary Jan 14, 2022
@vercel vercel locked as resolved and limited conversation to collaborators Feb 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants