Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Updated dependencies to resolve vulnerabilities #701

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

peter-stewart-thg
Copy link

@peter-stewart-thg peter-stewart-thg commented Dec 18, 2024

There were a number of outdated packages,
as well as many that had either been depricated or moved.

Some packages are no longer needed, as they were polyfilling
functionality that now exists later versions of JS or React.

I have replaced the use of 'react-loadable' with
Lazy and Suspense from React.

I've also migrated to the new Babel Mono-repo package structure.

I've updated the webpack config to resolve some issues with changes
in the config parameter as well and switching out
'extract-text-webpack-plugin' for 'mini-css-extract-plugin'
and 'eslint-loader' for 'eslint-webpack-plugin'

The issues related to this PR can be seen here: #700

UPDATE

I have just made a few more changes to this after testing the package in our WordPress install.
Unfortunately the changes are not quite functioning as intended.

I have just update how "react-select/async" and ajax from "rxjs" are used to bring them inline with the latests packages, however, it's still not working perfectly.

I might not have any more time to work on this, so if anyone else is able to take this on, please let me know.

There were a number of outdated packages,
as well as many that had either been depricated or moved.

Some packages are no longer needed, as they were polyfilling
functionality that now exists later versions of JS or React.

I have replaced the use of 'react-loadable' with
Lazy and Suspense from React.

I've also migrated to the new Babel Mono-repo package structure.

I've updated the webpack config to resolve some issues with changes
in the config parameter as well and switching out
'extract-text-webpack-plugin' for 'mini-css-extract-plugin'
and 'eslint-loader' for 'eslint-webpack-plugin'

I haven't currently resolve all issue with the build process,
however, the packages will now all install.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant