react-notification-system icon indicating copy to clipboard operation
react-notification-system copied to clipboard

library incompatible with React 17

Open detrash08 opened this issue 5 years ago • 5 comments

I've recently updated my project to react 17.0.2 and seems that the library is not compatible. Are you planning to update compatibility to React 17?

I have react-notification-system updated to verson 0.4.0

detrash08 avatar Apr 07 '21 10:04 detrash08

Was also curious about this, as we are looking to move to React 17 in the near future

chrisgbaker avatar Aug 18 '21 13:08 chrisgbaker

@detrash08 for the workaround, I used 0.2.17 and was able to resolve the issue with React 17

oleksandrbyk avatar Jan 07 '22 13:01 oleksandrbyk

@detrash08 for the workaround, I used 0.2.17 and was able to resolve the issue with React 17

The peer-dep will force you to install 2 React versions. (one for 16 and one for 17)

cristian-sima avatar Jan 28 '22 16:01 cristian-sima

I tried it and it works. You have to delete the peerDep from package.json and also to add this dep create-react-class

cristian-sima avatar Jan 28 '22 17:01 cristian-sima

I tried it and it works. You have to delete the peerDep from package.json and also to add this dep create-react-class

So you delete the perDeep Object ? and this dep where did you add it ? to the package json of the react notification or your entire project packafge.json ? and do you get the problem what i get now , i try to call the fork version, i get a problem said module cannot resolve react redux notification system and redux ? can please write me the step what u did to make it work and how you call the forked version to your package.json ?

Oskku avatar Oct 29 '22 13:10 Oskku