simplQ-frontend icon indicating copy to clipboard operation
simplQ-frontend copied to clipboard

Package version upgrades

Open maaverik opened this issue 4 years ago • 1 comments

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:

  1. Updated packages - https://nodejs.dev/learn/update-all-the-nodejs-dependencies-to-their-latest-version Removed react-qr-scanner package due to incompatibility with version of react-scripts
  2. Running dev server failed due to a gulp dependency error - fixed with https://stackoverflow.com/a/60921145/10281310
  3. Module not found error came up for react-refresh - fixed by changing versions of react-refresh-webpack-plugin as described in https://github.com/storybookjs/storybook/issues/17049 and installing react-refresh package
  4. react-typeform-embed export not found - fixed by replacing existing component with Popup https://www.npmjs.com/package/react-typeform-embed - still work in progress to fix an error
  5. React-router v6 has different interfaces for routing - made changes according to https://stackoverflow.com/a/69849271/10281310, https://stackoverflow.com/questions/62861269/attempted-import-error-usehistory-is-not-exported-from-react-router-dom, https://stackoverflow.com/a/69796015/10281310
  6. babel-eslint not found in .eslintrc - replaced with @babel/eslint-parser
  7. eslint throwing errors in browser instead of warnings in console, fixed according to https://stackoverflow.com/a/64630234/10281310

To Do:

  1. Add a working qr scanner module
  2. Fix broken lint rules
  3. Fix console errors

maaverik avatar Dec 31 '21 15:12 maaverik

Ah. M1 pains!

On Fri, 31 Dec, 2021, 9:00 pm Nithin, @.***> wrote:

@maaverik https://github.com/maaverik requested your review on: #686 https://github.com/SimplQ/simplQ-frontend/pull/686 Package version upgrades.

— Reply to this email directly, view it on GitHub https://github.com/SimplQ/simplQ-frontend/pull/686#event-5830685709, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACV4EOFLXNOWJGM6HC2Q5TTUTXD75ANCNFSM5LBNOFLQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because your review was requested.Message ID: @.***>

daltonfury42 avatar Dec 31 '21 17:12 daltonfury42