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

Angular2Parse dependency prevents upgrade to angular 10/11

Open renegadeCode opened this issue 4 years ago • 3 comments

Not sure what can be done. I tried to do a PR fixing the issue angular2parse has with newer versions of Angular.

Intended outcome: Upgrade angular to 10/11.

Actual outcome: Failed upgrade because of angular-cesium's dependency on angular2parse.

How to reproduce the issue: Use ng to upgrade angular to angular 11.

Version

renegadeCode avatar Dec 28 '20 17:12 renegadeCode

I forced the upgrade on angular 11 (didn't really checked this specific issue) and I am running with it without a problem. It will potentially break in the future, but for a version bump to latest angular it is ok for the moment.

Crocsx avatar Feb 13 '21 10:02 Crocsx

Cool, You mean this fork? : https://github.com/Crocsx/angular-cesium are there any plans for merging this into the original code base?

Or @Crocsx Maybe you can create a PR

aznguy avatar Mar 30 '21 12:03 aznguy

@aznguy No, this original repo works on angular 11 without issue. (I mean at least for what I use it) Only need to add the

"allowedCommonJsDependencies": ["geodesy", "util"],

to remove the compilation warnings.

I think you can try to force the update if you need.

My fork was a tentative to make this library works with the Cesium provided types (without having Cesium attached to the window object) and remove obsolete packages, but I kind of give up for now. ^^

Crocsx avatar Mar 30 '21 13:03 Crocsx