The PR #223 updated all the project dependencies which broke the build process and many other stuff. The reason is that a lot of CRA files needed to be updated as well.
updated config/ but kept stylus and rollbar configs
updated scripts/
updated package.json
fixed tests
fixed dynamic imports usage
fixed error handling in Home component
What needs to be tested:
[ ] Run the app in development and see if it works correctly in all compatible browsers
[ ] Build the project and run the app in production and see if it works correctly in all compatible browsers (you may use a local static server for this)
[ ] Test a version deployed via the CI to see if everything works correctly
[ ] Check if Rollbar is still working as expected, including the source-maps
It's important dedicate some time in testing all the updated stuff. More specifically, there were packages that were updated to new major versions.
What needs to be done next (create issues for these):
[ ] Fix linting errors
[ ] Integrate standard linting into CRA instead of using eslint
[ ] npm run test:e2e does not run anything; this is happening in master
@pgte feels free to close this and cherry-pick the commit to #223.
The PR #223 updated all the project dependencies which broke the build process and many other stuff. The reason is that a lot of CRA files needed to be updated as well.
What needs to be tested:
It's important dedicate some time in testing all the updated stuff. More specifically, there were packages that were updated to new major versions.
What needs to be done next (create issues for these):
npm run test:e2e
does not run anything; this is happening in master@pgte feels free to close this and cherry-pick the commit to #223.