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

docs(cloudflare): fix the environment variable howto #111

Merged
merged 1 commit into from
Mar 27, 2025
Merged

docs(cloudflare): fix the environment variable howto #111

merged 1 commit into from
Mar 27, 2025

Conversation

vicb
Copy link
Contributor

@vicb vicb commented Mar 27, 2025

We previously told users to:

  • add NEXT_PUBLIC_... vars to the build vars
  • add non-NEXT_PUBLIC_... vars to the runtime vars

But:

  • non-NEXT_PUBLIC_... vars can be accessed at build time (i.e. for SSG) so they also need to be populated in the build vars
  • NEXT_PUBLIC_... vars are also added to the runtime vars on Vercel, so they also need to be populated in the runtime vars

@vicb vicb requested review from james-elicx and conico974 March 27, 2025 09:20
@vicb
Copy link
Contributor Author

vicb commented Mar 27, 2025

Thanks for the reviews!

@vicb vicb merged commit ac2b0cf into main Mar 27, 2025
1 check passed
@vicb vicb deleted the vicb/ev branch March 27, 2025 10:35
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.

3 participants