clemsonMakerspace / unified-makerspace

The Makerspace - now unified.
Apache License 2.0
9 stars 11 forks source link

Update NPM Dependencies #72

Closed JDaniel41 closed 2 years ago

JDaniel41 commented 2 years ago

Notes

Ran npm update to update the dependencies and address the concerns brought by dependabot. Specifically, the following dependencies were updated:

Testing

I ran npm start to deploy a developer version of the site on my local machine. I was able to switch between the different versions of the site for each location, as well as type in the usernames. Anything that did not involve hitting the lambda backend was successful, as trying to hit the lambda results in CORS errors.

ophilli commented 2 years ago

Until we implement #62 the React App needs to be manually built & the build artifacts need to be checked into cdk/visit/console/. Currently this PR updates the source, but won't impact the deployed website.

Can you build the app, commit the build artifact changes, and then add that commit to this PR?