search

reactjs / react-chartjs

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

React 0.14 #54

Closed oliger closed 8 years ago

oliger commented 8 years ago

Same as https://github.com/jhudson8/react-chartjs/pull/48 but added react-dom in peerDependencies.

Anahkiasen commented 8 years ago

Could this be merged and tagged? Currently react-chartjs is the only one of the libs I use that still throws warnings in 0.14

eanplatter commented 8 years ago

Same here, +1

mrpapercut commented 8 years ago

Same here, waiting for this pull request to be merged

gregthebusker commented 8 years ago

This project doesn't seem to be maintained any more. You can use this package instead, which includes this pull request. https://www.npmjs.com/package/rc-chartjs

mrpapercut commented 8 years ago

@gregthebusker will you be actively maintaining the package (keeping up to date with new react releases, eliminating bugs), actively developing (adding new features etc) or is it just going to be the current version, including pull request?

gregthebusker commented 8 years ago

It depends what you mean by active. I'll be actively making sure it's up to date with the most recent version of react, but I'm going to rely on pull requests for bug and new features.

On Tue, Nov 3, 2015, 12:50 AM Mischa Rodermond notifications@github.com wrote:

@gregthebusker https://github.com/gregthebusker will you be actively maintaining the package (keeping up to date with new react releases, eliminating bugs), actively developing (adding new features etc) or is it just going to be the current version, including pull request?

— Reply to this email directly or view it on GitHub https://github.com/jhudson8/react-chartjs/pull/54#issuecomment-153289832 .

mrpapercut commented 8 years ago

Alright, thanks for the reply!

austinpray commented 8 years ago

@jhudson8 would you consider adding some more contributors to the repo?

jhudson8 commented 8 years ago

@austinpray I've added you. Thanks for helping out