angular-dc icon indicating copy to clipboard operation
angular-dc copied to clipboard

lodash 3.5

Open ruemic opened this issue 9 years ago • 5 comments

Getting this error when upgrading to lodash 3.5

dc.errors.InvalidStateException {message: "Mandatory attribute chart.dimension is missing on chart[#dc-chart1]", toString: function}angular.js?body=1:9938 (anonymous function)angular.js?body=1:7284 $getangular.js?body=1:12415 $get.Scope.$digestangular.js?body=1:12661 $get.Scope.$applyangular.js?body=1:8273 doneangular.js?body=1:8478 completeRequestangular.js?body=1:8417 xhr.onreadystatechange

ruemic avatar Mar 17 '15 23:03 ruemic

Had to roll back to lodash 2.4.1 to get this working.

jgknott avatar Mar 23 '15 20:03 jgknott

There is a pull request for this error at #34, not merged yet, though. @TomNeyland Were you able to review this request?

smee avatar Apr 14 '15 10:04 smee

Has this issue been resolved? Receiving the same error with lodash 3.10.1.

toddlamothe avatar Aug 14 '15 21:08 toddlamothe

I'm not a huge fan of being cornered into choosing the version of lodash that I use based on a dependency.

Thoughts on removing lodash as a dependency of this project? Maybe just implement the parts needed to not force others to have to use the same version of lodash?

Any other alternatives I'm not thinking about?

dynajoe avatar Oct 26 '15 17:10 dynajoe

The lodash incompatibility is resolved by the #34 patch cited above. The other lodash calls in angular-dc are common, stable and appropriate. I suggest that once the patch is merged, the bower lodash dependency be changed from '~2' to '*'. It is highly unlikely that the remaining lodash functions used by angular-dc would change. However, it is advisable to add a comment for contributors listing the allowed lodash functions.

FredLoney avatar Mar 30 '16 18:03 FredLoney