DOI-USGS / temperature-prediction

Other
7 stars 10 forks source link

Update node #280

Closed hcorson-dosch-usgs closed 1 year ago

hcorson-dosch-usgs commented 1 year ago

This PR updates our npm configuration to use node version 20 and re-point the docker build to debian:stable, because debian:stretch has been deprecated.

To test

  1. Get off VPN if you are on VPN
  2. Check what version of node you are using by running node -v
  3. If not using version 20, switch to node version 20 by running nvm use 20 [If it's not installed, install latest version of node by running nvm install 20.0.0 --reinstall-packages-from=14.17.0]
  4. Run npm install
  5. Run npm run serve

Changes:


First . . .

Then check for accessibly compliance

Then run Browserstack; check that application works on . . .

Finally . . .

After making a pull request

hcorson-dosch-usgs commented 1 year ago

This worked. Thank you.

No prob!

So the env NODE_OPTIONS=--openssl-legacy-provider piece is related to versioning changes b/w node and webpack?

Right - sounds like node changed to a later version of OpenSSL, which broke some part of webpack, and you have to use this workaround. It looked like webpack wasn't included in this package.json, but I opted to include the workaround just in case and so that we have a consistent set-up across our builds.