Replace deprecated dependency react-addons-css-transition-group

[punya]

Bug report

• Package version(s): 1.16.0
• Browser and OS versions: Chrome 57 on OSX (but it doesn't matter)

Steps to reproduce

1. Install Blueprint using YARN

Actual behavior

Requires the deprecated react-addons-css-transition-group package as a peer dependency.

Expected behavior

Allows (preferably, recommends) the react-transition-group fork, as recommended by the README for react-addons-css-transition-group.

[bensalilijames]

Are you accepting PRs for this? Would quite happily fix!

[llorca]

@benhjames for sure, we'd appreciate a contrib!

[remziduzagac]

Im still having this error with 1.17.1 version. is that normal?

[bensalilijames]

Yeah - it's a breaking change so it was decided to release with 2.0.0 instead - the patch 1.17.1 reverted the fix. 🙂

[remziduzagac]

☺ thank you very much 23 May 2017 18:55 tarihinde "Ben James" <[email protected]> yazdı:

…

Yeah - it's a breaking change so it was decided to release with 2.0.0 instead - the patch 1.17.1 reverted the fix. 🙂 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1051 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACjtZdsgeatAnTjST4xH7UpNTDtwyVdvks5r8wF6gaJpZM4NMGxF> .

[tangert]

is this still open? dealing with this error right now, trying to figure it out. has an update been made that fixes the break?

[rohmanhm]

Anybody would like make a PR I'll appreciated.

[bensalilijames]

The PR that fixed this was reverted since it is a breaking change, and will likely release with v2.0.0 of Blueprint. :)

[mkmpvtltd1]

i have same issue how can i fix this?

[giladgray]

@mkmpvtltd1 ignore it! It's not an error, just a casual warning. Your app will continue to function perfectly despite the outdated library.

We'll bump all our dependencies later this year.

[jsina]

I've get the same error
"Uncaught Error: Cannot find module "react-addons-css-transition-group"

here is my package.json

"dependencies": {
    "@blueprintjs/core": "^1.27.0",
...

[ChisholmKyle]

I'm trying to upgrade to React 16. If I try to build without the react-addons-css-transition-group dependency, I get this error when running browserify:

Error: Cannot find module 'react-addons-css-transition-group' from '/path/to/project/node_modules/@blueprintjs/core/dist/components/overlay'

So I try to install "react-addons-css-transition-group" but the module does not support React 16. However if I try to install an alpha version compatible with React 16 as follows:

npm install --save-dev react-addons-css-transition-group@^16.0.0-alpha

I get an error that it's incompatible with Blueprintjs:

npm ERR! peerinvalid The package [email protected] does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer @blueprintjs/[email protected] wants react-addons-css-transition-group@^15.0.1 || ^0.14

Is it possible to use blueprintjs v1.x with React 16? I thought there was an update recently to at least try it out.

[giladgray]

@ChisholmKyle in my testing, it seemed that react-addons-css-transition-group@15 totally supports React 16! I wasn't even aware there was an alpha, or that such a thing would be necessary (pretty sure it isn't).

it's unsupported by the React dev team (hence the new package), but the existing API is still fully functional. so you should be able to install it alongside Blueprint using a version that matches our peerDependency.

either way, this issue will be like the first thing we do when we begin work on 2.0 later this year.

[giladgray]

@jsina you must declare your own dependencies on react, react-dom, and react-addons-css-transition-group because we declare them as peerDependencies. https://nodejs.org/en/blog/npm/peer-dependencies/

[ChisholmKyle]

I found out my problem. I'm using typescript and it was actually the package @types/pure-render-decorator causing the problem. When I removed that dependency from package.json, React 16 is now working with blueprint.js and react-addons-css-transition-group!

Thanks for your help.