-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
It doesn't support angular ivy mode #4236
Comments
@wenqi73 Would you please take a look at this? Be aware of potentional breaking changes. |
@wendzhue this is the same problem I had seen weeks ago when the beta started. Anyway, @justwalk there is settings file named |
@justwalk #3514 (comment) blocking issues in Angular sadly for Ivy support to happen now |
@lppedd you can try compiling with Ivy enabled now with Angular version 9.0.0-rc.1. Haven't tested it myself yet, but all the blocking issues seem to be closed now. |
@Svetomechc oh ok. Thanks! Even the base classes decorators has been solved? |
@lppedd I hope. |
@Svetomechc I haven't seen commits about that here, but I'll try anyway. |
@lppedd have you had any success? |
angular/angular#33393 there is still some issue related to ivy |
@vthinkxie added thumbs up to the issue, hope it gets resolved ASAP. |
@Svetomechc that issue is now on |
I'm pretty much just waiting for this, since we've switched over to Bazel and this is blocking us from getting any further. |
Closed by #4453. We would release a beta version that supports Ivy shortly after. Thanks for your patience and waiting. ❤️ |
Awesome! I just finished integrating view engine.. guess I'm gonna go back to Ivy again then, cheers! |
Seems like that didn't resolve the issue 😅
|
@marcus-sa Have you tried to remove node_modules and reinstall? Usually this problem comes from an outdated version of @angular/cdk. |
@marcus-sa I succesfully enabled Ivy with ng-zorro-antd, ping me if you need any help. |
@marcus-sa make sure you updated angular, zone.js, typescript, etc.: make sure you have this line in angular.json: not sure if this is still needed, but here's my tsconfig.app.json: delete your package-lock.json / yarn.lock files, remove node_modules directory, run npm i / yarn commands then your app should run smoothly (first run will take longer to compile. as it will compile your node_modules packages to new format) |
@Svetomechc thanks, but it's weird because I have the exact same versions of the packages installed and it still doesn't work. EDIT: |
Worked like charm all I did was to follow the official documentation of upgrading to angular 9 here |
It no longer works with
Which results in the following error when trying to import node_modules/ng-zorro-antd/ng-zorro-antd.module.d.ts:72:22 - error NG6003: Appears in the NgModule.exports of SharedModule, but could not be resolved to an NgModule, Component, Directive, or Pipe class
72 export declare class NgZorroAntdModule {
~~~~~~~~~~~~~~~~~
|
Hi all, we are working on the ng-zorro-antd v9 now. anyone who has interest please follow this issue #4634 I believe it would release soon after the angular v9 final version. |
The problem still exis when ngcc, it will thrown an error like this:
but it could be solved by delete node_modules and then reinstall |
It's not still safe to use ng zorro with angular 9 ? |
We will release stable version(9.0.0) today |
Hi, with the version 9 and Angular 9 (ivy enabled) I am now getting the following error: And then I get a bunch of other errors all related to nz-no-animation.directive.js I tried removing node_modules folder and running a clean install but when i run ngcc i get the same result. That is a really big blocker for me. Can you please advise? |
Is ngcc required to run? I think its more of a good thing to do for faster build time ? |
Unfortunately I get the same errors also during build and running unit tests while the dependencies are compiled so despite the unit tests passing the pipelines are failing due to these errors. |
Has the problem been solved? This error is still thrown when I run ngcc after upgrading to 9: |
@zmzimpl It will be fixed in the next version(9.0.2), you can subscribe to the release notifications |
OK,thanks,It also seems to throw this error: |
@zmzimpl Looks like a common error on ivy, could you delete the |
I tried deleting |
What problem does this feature solve?
it can use angular ivy mode to improve application performance.
What does the proposed API look like?
Ng-zorro cannot compile through ivy mode because date-fns does not support Angular Package Format。
The text was updated successfully, but these errors were encountered: