Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
npm install was failing for me on a new OS with higher node version (v16), so I thought I'd try to update packages to get it to install and fix vulnerabilities along the way. npm reported >100 vulnerabilities with existing versions.
This PR need not be merged unless we absolutely have to upgrade versions. It's only partially been tested, so bugs probably remain. I successfully updated most modules, and got the vulnerabilities reduced to 37. I had to uninstall the qr-scanner module, but it wasn’t working properly anyway, some console errors still remain, and eslint reports some false positive lint errors. I'll get back to fixing this sometime, but just documenting what I did here in case the same thing has to ever be done again someday.
Steps so far:
Removed react-qr-scanner package due to incompatibility with version of react-scripts
To Do: