-
-
Notifications
You must be signed in to change notification settings - Fork 241
@nativescript/angular does not compile with @nativescript/core #2058
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
Comments
is the tns-core-modules deprecated at version 6.x.x? |
I am getting this too. |
@hamidbsd @davecoffin @atchaiecommerce this was indeed a bug that is addressed in the latest |
@NickIliev thanks for weighing in! I removed tns-core-modules and nativescript-angular from my deps, was that the right move?
I am running the 6.3 cli, would that also cause a problem? |
In other news, I tried upgrading to 6.3.1 by removing @nativescript/angular and @nativescript/core from my deps, and adding this:
but now I get this error:
|
ok i finally got this figured out by upgrading to 6.3.1, removing But I guess my question @NickIliev is...how can i use the |
I spoke too soon, it built but when running using aot i get this error when the app launched: |
ok i resolved this finally but am not entirely sure how haha. a word of warning: dont import both |
@NickIliev , this is still a problem, even with the latest 6.3 packages. I just created a brand new NG app replaced all the deps to the scoped packages. Running w/o AoT compiles and runs the app just fine. Enabling AoT throws the error in the OP 😞 |
@PeterStaev are you importing |
@davecoffin , this is happening with a brand new project created with the 6.3 CLI that had all references updated to use the scoped packages. So there is no |
right, so i think this may be a problem with importing |
@PeterStaev To be clear, keep importing from the barrel everywhere, just change to |
So we still have to have BOTH This is not a production app, but a demo app for one of my plugins, so wont do anything. I already wrote in the release that there is a known issue and users will get an error if they use AoT and linked to this issue, so it is all in the hands of the NS Core Team 🙃 |
Actually you cant have both as deps, because each pulls in tns-core-modules-widgets, so if you have both you get an error that TNSWidget is created in two places. |
Hi guys, Thanks for your investigation and sorry for the late response. We've merged a fix in the @davecoffin you can use both |
Fix released with |
It fails with |
Environment
{N} 6.2
Describe the bug
Webpack compilation breaks with the following error
Expected behavior
To compile without any problems
Additional context
This file is causing the problem as it still tries to import from tns-core-modules instead of @nativescript/core.
nativescript-angular/nativescript-angular/platform-providers.ts
Lines 3 to 6 in 87c8cea
The text was updated successfully, but these errors were encountered: