redux-injectors icon indicating copy to clipboard operation
redux-injectors copied to clipboard

Unmet peer dependencies with React 18

Open nixjs opened this issue 3 years ago • 4 comments

I have a issues when use redux-injectors with React ^18.

image

nixjs avatar Jun 03 '22 03:06 nixjs

FWIW I've just tested 2.0.0 with React 18 and react-redux 8.0.1, and things seem to be working fine AFAICT. Possibly React 18 compatibility is a no-op (and just needs the peer dependencies to be bumped), but I don't think I use all the advanced features of redux-injectors, so grain of salt.

abesto avatar Jun 03 '22 09:06 abesto

@nixjs did u solve this issue?

avx99 avatar Sep 25 '22 09:09 avx99

@nixjs did u solve this issue?

i still have the above problem

nixjs avatar Sep 26 '22 05:09 nixjs

@nixjs did u solve this issue?

I think update package.json in redux-injectors:

"peerDependencies": {
    "react": "^16.6.0 || ^17.0.0 || ^18.0.0",
    "react-dom": "^16.6.0 || ^17.0.0 || ^18.0.0"
  }

instead of:

"peerDependencies": {
    "react": "^16.6.0 || ^17.0.0",
    "react-dom": "^16.6.0 || ^17.0.0"
  }

The problem will be solved.

nixjs avatar Sep 26 '22 06:09 nixjs

When do you upgrade new version to support react v18?

nixjs avatar Oct 18 '22 09:10 nixjs

When do you upgrade new version to support react v18?

I forked and supported react v18: https://www.npmjs.com/package/@nixjs23n6/redux-injectors

nixjs avatar Nov 09 '22 03:11 nixjs

@nixjs Nice Job! I hope some day redux-injectors do something like you did in your fork, basically using > and < between versions.

carmenates09 avatar Nov 18 '22 17:11 carmenates09

This is fixed in 2.1.0

BenLorantfy avatar Nov 22 '22 13:11 BenLorantfy