Open SeanSobey opened 2 years ago
That sounds fine to me
Really glad I found this, was ready to give up.
idk what I'm doing but I added to Sean's fix and pasted into chartjs-chart-financial.min.js
and now I have pretty candlesticks saved from node :))
...afterBuildTicks:e=>{if(typeof(this)=='undefined'){return;}else{if(!('window' in this)){return;};};const t=window&&window.luxon&&window.luxon.DateTime;...
Created a pull request for this
While running this plugin on NodeJS we are getting this error (running ChartJS via my lib):
Obviously, the error is from the
window
reference. It looks to just be the one reference, when checking for Luxon, and it can be resolved via something like this:I am happy to test and submit this via pull request but am just inquiring about interest and appetite to support NodeJS with this fix and going forward?