Skip to content

Use react-transition-group from npm#9937

Merged
gaearon merged 1 commit into
react:15.6-devfrom
gaearon:use-transition-group
Jun 12, 2017
Merged

Use react-transition-group from npm#9937
gaearon merged 1 commit into
react:15.6-devfrom
gaearon:use-transition-group

Conversation

@gaearon

@gaearon gaearon commented Jun 12, 2017

Copy link
Copy Markdown
Collaborator

Note: I have not tested this yet.

This should point react-addons-transition-group to API-compatible versions of react-transition-group package. This lets it keep working in observable future. So we don't need to deprecate it officially (even though we will keep the npm package deprecated and suggest people to migrate to new package explicitly).

cc @jquense

@flarnie

flarnie commented Jun 12, 2017

Copy link
Copy Markdown
Contributor

If/when I have time I can come back to this and manually test it. I think we could use the 'browserify' fixture and require and use it there to manually test.

@gaearon

gaearon commented Jun 12, 2017

Copy link
Copy Markdown
Collaborator Author

I'm going to assume it works—I'll do another round of testing anyway as I complete preparing this, and I'd rather test them all at once.

@gaearon gaearon merged commit dea8f6a into react:15.6-dev Jun 12, 2017
@gaearon gaearon deleted the use-transition-group branch June 12, 2017 22:43
@jquense

jquense commented Jun 13, 2017

Copy link
Copy Markdown
Contributor

Looks like it should work to me...i don't think I changed the exports at all 👍

nhunzaker pushed a commit to nhunzaker/react that referenced this pull request Jun 13, 2017
@flarnie flarnie mentioned this pull request Jun 13, 2017
49 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants