Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Make react-charts compatible with react@0.14.0-rc1 #48

Closed
wants to merge 1 commit into from

Conversation

dmr
Copy link
Contributor

@dmr dmr commented Sep 20, 2015

No description provided.

@dmr dmr mentioned this pull request Sep 20, 2015
@dmr
Copy link
Contributor Author

dmr commented Sep 20, 2015

But react 0.14 isn't released yet and it might break compatibility so I'm not sure if this should be merged yet

@dmr
Copy link
Contributor Author

dmr commented Sep 20, 2015

package.json refers a "test" command but I don't see any tests

oliger added a commit to hull/react-chartjs that referenced this pull request Oct 9, 2015
Make react-charts compatible with react@0.14.0-rc1
@oliger oliger mentioned this pull request Oct 9, 2015
@gustavoguichard
Copy link

I think this should be merged now because react 0.14 was released already.
And also, getDOMNode() was already deprecated in react 0.13.

@Anahkiasen
Copy link

👍

@santiagoaguiar
Copy link

+1 needing this one to move forward to 0.14.

@tomfairbs
Copy link

Yep, +1 from me as well. Would also be a good idea to update the compiled versions in /dist.

@sspross
Copy link

sspross commented Oct 19, 2015

+1

1 similar comment
@siemya
Copy link

siemya commented Oct 21, 2015

+1

@dmr
Copy link
Contributor Author

dmr commented Oct 23, 2015

Do I read correctly that this pull request is outdated and peerDependencies are the way to got and #54 already added all the things that I didn't?

@eanplatter
Copy link

@dmr yeah, it looks like #54 covers it all.

@dmr dmr closed this Oct 24, 2015
@dmr
Copy link
Contributor Author

dmr commented Oct 24, 2015

Well, then we can just close this :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants