react-localize-redux icon indicating copy to clipboard operation
react-localize-redux copied to clipboard

Update react peer dependency

Open cristianocca opened this issue 3 years ago • 4 comments

Do you want to request a feature or report a bug? bug

What is the current behavior? Localize cannot be installed with NPM >= 7 due to peer dependency issues. Please allow react 17 in this library's peer deps.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code. Paste the link to your JSFiddle (https://jsfiddle.net/Luktwrdm/) or CodeSandbox (https://codesandbox.io/s/new) example below:

npm ERR! code ERESOLVE npm ERR! ERESOLVE unable to resolve dependency tree npm ERR! npm ERR! While resolving: XXX npm ERR! Found: [email protected] npm ERR! node_modules/react npm ERR! react@"17.0.2" from the root project npm ERR! npm ERR! Could not resolve dependency: npm ERR! peer react@"^16.0.0" from [email protected] npm ERR! node_modules/react-localize-redux npm ERR! react-localize-redux@"3.5.3" from the root project npm ERR! npm ERR! Fix the upstream dependency conflict, or retry npm ERR! this command with --force, or --legacy-peer-deps npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

What is the expected behavior? No dependency issues

Which versions of react and react-localize-redux are you using? 3.5.3

cristianocca avatar Sep 20 '21 18:09 cristianocca

@ryandrewjohnson any chance for this to be updaded? Are you no longer maintaining the library?

Thanks!

cristianoccazinsp avatar Oct 29 '21 16:10 cristianoccazinsp

I had been working on updating the library a while back but I just haven’t had the time unfortunately. At this point it’s not being actively maintained. There is a branch created for v4 if anyone wanted to ever try and pick it up.

ryandrewjohnson avatar Nov 18 '21 04:11 ryandrewjohnson

Hi @ryandrewjohnson , looks like the library now also depends on a vulnerable package and will flag it as vulnerable (https://github.com/ryandrewjohnson/react-localize-redux/issues/235). I wonder if it would be possible to at least bump the dependencies. Are there any breaking changes or major code changes required? I may be willing to pick this up if it's just patching some packages.

cristianoccazinsp avatar May 09 '22 22:05 cristianoccazinsp

Hello, any updates around this? Any new fork?

cristianoccazinsp avatar Aug 12 '22 15:08 cristianoccazinsp