search

reactjs / react-chartjs

common react charting components using chart.js
MIT License
2.93k stars 301 forks source link

Updated for React 0.14, fixed when updating labels #59

Closed zackify closed 8 years ago

zackify commented 8 years ago

Some of your code isn't in the latest version on npm, like nextProps.redraw is this.props.redraw on npm, spent 30 minutes figuring that out lol. This should fix everything.

austinpray commented 8 years ago

Isn't this forgetting to add react-dom as a peerDependency?

zackify commented 8 years ago

@austinpray you're right. After making this PR I saw how there is no activity on this repo and forgot about it.

sspross commented 8 years ago

öhm so what's going on here? things are fixed but v0.7.1 is no available on npm. how can we help?

joshhornby commented 8 years ago

That was my mistake, I have released 0.7.2 https://github.com/jhudson8/react-chartjs/tree/0.7.2.

joshhornby commented 8 years ago

@jhudson8 Can you please update NPM. Thanks

sspross commented 8 years ago

@jhudson8 is there still hope or should we switch over to https://github.com/gregthebusker/rc-chartjs?