search

pablojim / highcharts-ng

AngularJS directive for Highcharts
MIT License
1.73k stars 463 forks source link

Update highcharts-ng.js #635

Closed lwallent closed 6 years ago

lwallent commented 6 years ago

Hi @rafaelmacario and @pablojim

This PR will fix a bug introduced in fixing issue #592 (please see my comments to that issue)

KR Lasse

pablojim commented 6 years ago

@lwallent Thank you!

lwallent commented 6 years ago

Hi @pablojim,

Any chance that my change will be released to the npm repository in the near future? That way I can keep my dependencies simple 👍 (and the current npm version fails when importing) Thanks for the great work!

KR

Lasse

kevinboosten commented 6 years ago

@pablojim a new release would be helpful :)

andrewboni commented 6 years ago

@lwallent thank you! I also encountered this same issue and spent a lot more time than I'd have liked trying to resolve

@pablojim gentle bump on a new release :)