Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade dependency to new version of Chart.js #41

Open
NateRedding opened this issue Jul 10, 2014 · 5 comments
Open

Upgrade dependency to new version of Chart.js #41

NateRedding opened this issue Jul 10, 2014 · 5 comments

Comments

@NateRedding
Copy link
Contributor

There is a new pre-release version of Chart.js. It looks like they are changing the reference in bower.json to 'Chart.js' from 'nnnick-chartjs'. There are other plugins that reference using the new name as well, ie. https://github.com/Regaddi/Chart.StackedBar.js/blob/master/bower.json

Angles should be updated as well for consistency. (At least once Chart.js 1.0 is finalized.)

@DeepAnchor
Copy link

+1!! Woo, looks like nnick is back and he decided to add tooltip functionality!

@jmtt89
Copy link
Contributor

jmtt89 commented Jul 24, 2014

ready, now angles work with CharJS Beta

@NateRedding
Copy link
Contributor Author

In bower.json, you should refer to Chart.js using the identifier 'Chart.js' instead of 'chartjs'. With the 1.0 release, this seems to be the recommended Bower identifier as other projects that I know work closely with Chart.js refer to it this way.

If you use a different identifiers, people end up with two copies of Chart.js thanks to Bower not realizing they are the same project.

@LeleDev
Copy link

LeleDev commented Feb 4, 2015

+1... fix it soon please

@jmelosegui
Copy link

👍 and Dup from #57

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

No branches or pull requests

5 participants