chartjs-chart-financial.js intergrate

mariusmuntean / ChartJs.Blazor

Brings Chart.js charts to Blazor

https://www.iheartblazor.com/

MIT License

677 stars 151 forks source link

chartjs-chart-financial.js intergrate #72

Open Jong-uk-Kim opened 4 years ago

Jong-uk-Kim commented 4 years ago

issue: chartjs-chart-financial.js interrte

implemented chartjs extension for financial.

※ Notes ChartJs.min.js verserion updated to v2.9.3, because financial extension has dependency over v2.9.0.

Joelius300 commented 4 years ago

I don't have much time currently so I can't review it yet, sorry. Also I'm hesitating about upgrading chart.js since it will mean changes in other places as well; we'll have to think that through and also quickly (which is not this repos best quality) update all the C# code to work with the new chart.js release. It shouldn't be that big of a deal, I'm probably overdramatic but as I said, I don't have much time at the moment :/

Jong-uk-Kim commented 4 years ago

ok, i understand your concern, if you can be a time then merge this. :)