Skip to content

move to external react-transition-group dependency#399

Merged
eddywashere merged 2 commits intoreactstrap:masterfrom
ened:react-transition-group
May 14, 2017
Merged

move to external react-transition-group dependency#399
eddywashere merged 2 commits intoreactstrap:masterfrom
ened:react-transition-group

Conversation

@ened
Copy link
Copy Markdown
Contributor

@ened ened commented Apr 20, 2017

Per https://facebook.github.io/react/blog/2017/04/07/react-v15.5.0.html:

react-addons-css-transition-group - Use react-transition-group/CSSTransitionGroup instead. Version 1.1.1 provides a drop-in replacement.

Please review the webpack.base.config.js changes whether it needs changes.

@ened ened force-pushed the react-transition-group branch from f53cff0 to 6fca069 Compare April 20, 2017 15:31
@ened
Copy link
Copy Markdown
Contributor Author

ened commented Apr 26, 2017

Would love to know if anyone is using that patch already?

@nathancahill
Copy link
Copy Markdown
Contributor

Yes, I tested it, but I use Rollup, so I can't speak for the Webpack changes. It works fine otherwise. Would love to see this merged soon, currently a blocker for upgrading to latest React.

@ened
Copy link
Copy Markdown
Contributor Author

ened commented Apr 27, 2017

@eddywashere Hi there, could you please give us an update? Thank you

@samrod
Copy link
Copy Markdown

samrod commented Apr 28, 2017

Please merge this PR ASAP. The deprecated dependencies are no longer installing and it's breaking our app.

Thanks,

@TheSharpieOne
Copy link
Copy Markdown
Member

@samrod that is not how deprecations work.

@ghost
Copy link
Copy Markdown

ghost commented May 2, 2017

Any status updates regarding this PR?

@ened
Copy link
Copy Markdown
Contributor Author

ened commented May 7, 2017

.oO( time to fork )

@eddywashere eddywashere merged commit a4fec3c into reactstrap:master May 14, 2017
@balloob
Copy link
Copy Markdown
Contributor

balloob commented Jun 22, 2017

This PR messed up the Rollup config, bumping the file size from 95kB to 335kB. Reported the issue as #472.

I have opened a PR with a fix in #474.

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.

6 participants