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 server components not reading env vars #70

Merged
merged 4 commits into from
Aug 27, 2023

Conversation

HofmannZ
Copy link
Member

@HofmannZ HofmannZ commented Aug 27, 2023

This PR fixes an issue in the app dir example:

By default, server components are statically generated at build time. To ensure the env vars are loaded, the server component needs to be forced to be dynamic.

Additionally, this PR upgrade the dependencies and splits out some internal functions and types.

Fixes #35

@codecov
Copy link

codecov bot commented Aug 27, 2023

Codecov Report

Merging #70 (4a1e9ef) into development (381bdf9) will not change coverage.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##           development       #70   +/-   ##
=============================================
  Coverage       100.00%   100.00%           
=============================================
  Files                8         9    +1     
  Lines               60        63    +3     
  Branches             9         9           
=============================================
+ Hits                60        63    +3     
Files Changed Coverage Δ
src/all-env.ts 100.00% <100.00%> (ø)
src/helpers/get-browser-env-script.ts 100.00% <100.00%> (ø)
src/helpers/write-browser-env.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@HofmannZ HofmannZ merged commit 02b254f into development Aug 27, 2023
8 checks passed
@HofmannZ HofmannZ deleted the @HofmannZ/fix/server-components branch August 27, 2023 17:46
@expatfile-bot
Copy link
Member

🎉 This PR is included in version 1.7.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@expatfile-bot
Copy link
Member

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@expatfile-bot
Copy link
Member

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@expatfile-bot
Copy link
Member

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants