-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I am trying to use wrld.js with Angular #148
Comments
More precisely, this is not something related to Angular, but to Typescript. If I have a simple TS file, named
when I run Probably the issue will be fixed if you update the |
Can you please try changing |
@chrisdjali-wrld3d Yes, it doesn't change anything, because the errors are caused by
However, the
but this is not something you want to do since those files will be changed back at each |
I suspect that whatever's causing this error is also causing all of the others. There is definitely supposed to be a default export, and it works on our end and for many of our customers, so we've not simply forgotten it. Can you think of anything potentially atypical about how you're consuming the wrld.js package? |
After some investigation, we've determined that wrld.js is intended to be used with the TypeScript compiler's |
@chrisdjali-wrld3d I didn't do anything atypical. I created a simple TypeScript app here in Stackblitz. Unfortunately, Stackblitz doesn't show errors from the libraries (i.e. wrld.js) from node_modules, so you have to download the app to see that TypeScript doesn't compile. Note that I've replaced One problem I see in your package.json file
However, you should update |
Changing We actually do set If you're averse to setting the compiler option, you could manually go through our emitted type declarations and change every instance of As an additional hurdle you'll need to overcome, the 1.9.3 release of |
I'm okay using |
While we can theoretically tolerate the version of Generally, you can't rely on other Leaflet plugins working with wrld.js. For some of them, the reason's fairly obvious, e.g. a new Layer needs to know how to project itself onto the 3D map. Other things work fine, but it can be better to think of wlrd.js as an alternative to Leaflet rather than just an addon, and set expectations accordingly when trying to integrate it into an application that already uses Leaflet. If you want to take the risk of using an unsupported Leaflet version anyway and absolutely can't stick to |
Thank you for your time, I really appreciate. I understand that the resources are limited, but freezing to a version has pretty many downsides. However, in our app |
Hello @alinmateut The issue should be resolved now within. Version you are able to install this version with he following using npm But do continue to use Sam |
I have this errors of wrld.js:
The text was updated successfully, but these errors were encountered: