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
ERROR Cannot read property 'id' of undefined #190
Comments
Good day! Are you sure this is comming from the library itself? Can you give me the filename and number of the line in which this error is occurring? |
@Alessandroinfo anything new here? |
@KingSora I can confirm the issue:
You'll get:
|
@FSDRE Thank you, I'll try to reproduce it as soon as I have time. Do you know wheter the CLI version is the cause.. (does it also happen with 8.x?) |
@KingSora Just tried it with the latest NG 8.3.25 and it works. We also have a full blown NG 8.2.14 project running with OS perfectly. So I think we have a 9.x incompatibility issue. Saw a couple libraries fail the last couple of days because NG 9 does so many things differently but most of the issues were not of a very complicated nature. I hope this right here is also an easy one 😄 |
@FSDRE Alright... I'll look into this! :) |
@Alessandroinfo @FSDRE could you try running the command |
@KingSora Interesting... I'll try that on monday. But is that a one time compile after installing the npm package or do I have to run it on every It really hurts that IE11 is still a thing for certain projects. Could just get rid of the browser property and smile 😄 |
@KingSora I can confirm that |
I removed the |
I run my entire build through babel anyways, so it shouldn't be an issue (I think). |
Yeah I'm not saying they should just remove that line in their project. I just left that comment there as input and for the next person stumbling upon this via google. Some people just need to get a working build out the door first and foremost. |
@laino @FSDRE the fix will probably look like this. The PR for #166 will be reverted in the future. (see this comment: angular/angular#36062 (comment) and the comment above) |
The issue is probably due to how the library is compiled. Switching to ng-packagr should fix it.
|
@TomTomB @FSDRE @laino @Alessandroinfo please try the new |
@KingSora Your fix does compile now but your package still requires Angular 7 as dev dependency. Ivy is not available on Angular 7, so all ng packages have to be migrated as well (e.g. |
@KatzeKahn The package requires Angular 7 as a peerDependency, but its also perfectly useable with higher Angular versions. I need to change that in the next release, youre right... The Example app is written with Angular 7 but since this is just a example it wasn't a hight priority of mine to update it to v9.. But I'll also change that in the future. Thanks! |
@KingSora I can confirm that it works in NG9 now. There are 2 warnings:
It tries to pull the sourcemaps from |
@FSDRE Thanks! Please open a separate issue for this, I'll take a look when I have time. |
Working like a charm on our end now. Thanks! |
Remove NgModule imported in any submodule of the application. I believe the Ionic Team needs to give more readable messages and tracing the source map better. |
You are the hero that we need, thanks you! |
I have the same issue and it happens when serving the initial project files. No new modules was added, No packages was installed. I did not know the source of the issue. |
@Ahmed-Hafez can you find any solution? |
@FuadEsgerov Unfortunately no, but a temporary solution can be made by uninstalling node modules and reinstalling them. If not solved, replacing the project files into another folder which might solve the issue temporarily. But as I told this is temporary solution and the issue almost will raise again after some time. |
rm -rf node_modules |
its from lib.es5.d.ts line number 1340 |
Hi.
I retrieve this error: ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'id' of undefined when i add the module OverlayscrollbarsModule.
I use it on Angular 9.
The text was updated successfully, but these errors were encountered: