fix: 🐛 use next/script for PublicEnvScript

expatfile / next-runtime-env

Next.js Runtime Environment Configuration - Populates your environment at runtime rather than build time.

https://www.npmjs.com/package/next-runtime-env

MIT License

458 stars 19 forks source link

fix: 🐛 use next/script for PublicEnvScript #126

Closed davevanhoorn closed 6 months ago

davevanhoorn commented 6 months ago

This PR:

uses the Script tag from next/script, instead of the native HTML script tag in the env-script.tsx file
it uses the beforeInteractive strategy, so runtime variables are available (Load the script before any Next.js code and before any page hydration occurs.)

Notes:

We are using this package the same as people here, and they've got the same hydration issues.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (65c21fe) to head (0614869). Report is 1 commits behind head on development.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## development #126 +/- ## ============================================= Coverage 100.00% 100.00% ============================================= Files 12 12 Lines 83 84 +1 Branches 12 12 ============================================= + Hits 83 84 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

expatfile-bot commented 6 months ago

:tada: This PR is included in version 3.2.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: