Skip to content
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

Build and run-time errors when using Ivy #2321

Closed
0-gravity opened this issue Jun 11, 2019 · 28 comments
Closed

Build and run-time errors when using Ivy #2321

0-gravity opened this issue Jun 11, 2019 · 28 comments
Assignees
Labels

Comments

@0-gravity
Copy link

Describe the bug
Angular 8 fails to build a Grid when Ivy is enabled with error message: ERROR in Cannot combine @input decorators with query decorators

To Reproduce
Steps to reproduce the behavior:

Used Angular cli - 8.0.2
1.) Create new project with flag: --enableIvy=true
2.) Add angular grid via: ng add @progress/kendo-angular-grid
3.) Import GridModule into AppModule
4.) Run command: ng serve

Expected behavior
Should pass a build

kendo-grid-compilation-failure@0.0.0 c:\Development\bugs\KendoGridCompilationFailure
+-- @angular-devkit/build-angular@0.800.2
+-- @angular/animations@8.0.0
+-- @angular/cli@8.0.2
+-- @angular/common@8.0.0
+-- @angular/compiler@8.0.0
+-- @angular/compiler-cli@8.0.0
+-- @angular/core@8.0.0
+-- @angular/forms@8.0.0
+-- @angular/language-service@8.0.0
+-- @angular/platform-browser@8.0.0
+-- @angular/platform-browser-dynamic@8.0.0
+-- @angular/router@8.0.0
+-- @progress/kendo-angular-buttons@4.4.2
+-- @progress/kendo-angular-dateinputs@3.7.4
+-- @progress/kendo-angular-dropdowns@3.5.6
+-- @progress/kendo-angular-excel-export@2.4.0
+-- @progress/kendo-angular-grid@3.14.4
+-- @progress/kendo-angular-inputs@5.2.2
+-- @progress/kendo-angular-intl@1.7.1
+-- @progress/kendo-angular-l10n@1.4.1
+-- @progress/kendo-angular-popup@2.6.0
+-- @progress/kendo-data-query@1.5.1
+-- @progress/kendo-drawing@1.5.12
+-- @progress/kendo-theme-default@3.6.1
+-- @types/jasmine@3.3.13
+-- @types/jasminewd2@2.0.6
+-- @types/node@8.9.5
+-- codelyzer@5.1.0
+-- jasmine-core@3.4.0
+-- jasmine-spec-reporter@4.2.1
+-- karma@4.1.0
+-- karma-chrome-launcher@2.2.0
+-- karma-coverage-istanbul-reporter@2.0.5
+-- karma-jasmine@2.0.1
+-- karma-jasmine-html-reporter@1.4.2
+-- protractor@5.4.2
+-- rxjs@6.4.0
+-- rxjs-compat@6.5.2
+-- ts-node@7.0.1
+-- tslib@1.10.0
+-- tslint@5.15.0
+-- typescript@3.4.5
`-- zone.js@0.9.1

@tsvetomir
Copy link
Member

Support for Ivy is not in just yet. We'll follow-up when more information is available.

@markacx
Copy link

markacx commented Jun 26, 2019

Any timeline for this

@tsvetomir
Copy link
Member

tsvetomir commented Jun 27, 2019

Ivy is a bit of a moving target at the moment — some fixes we applied earlier are now obsolete. We can't commit to official support until things settle down a bit.

Also, note that you should the next version of @angular/core when testing with Ivy, see guide.

@tsvetomir tsvetomir self-assigned this Jul 3, 2019
@tsvetomir
Copy link
Member

This effort is currently blocked by angular/angular#30080

@lormerod
Copy link

angular/angular#30080 is now closed. Is kendo now working correctly under ivy?

@tsvetomir
Copy link
Member

Not yet—we still have a few smaller issues to take care of. We're working together with the Angular team to resolve these. They've set up a project dedicated to validating backwards-compatibility with Ivy, see angular/ngcc-validation#386 for more details.

@MaklaCof MaklaCof mentioned this issue Oct 30, 2019
@tsvetomir tsvetomir changed the title Angular 8 fails to build a project with Grid module when Ivy is enabled Build and run-time errors when using Ivy Oct 30, 2019
@tsvetomir
Copy link
Member

Updating the issue title as Ivy support affects all components, not just the Grid.

@thaoula
Copy link

thaoula commented Nov 9, 2019

@tsvetomir so would you want us to log ivy build issues like the following -

DtsModuleScopeResolver.read(LocalizedMessagesDirective$3 from /Users/thaoula/Projects/Empower/jms-platform/jms-web/node_modules/@progress/kendo-angular-dateinputs/dist/fesm2015/index.js), but not a .d.ts file

Or should we wait until you guys finish with the angular team?

@tsvetomir
Copy link
Member

@thaoula sorry for the delay!

This issue has been fixed in the development builds of the Date Inputs. Update by running npm install --save @progress/kendo-angular-dateinputs@dev or yarn add @progress/kendo-angular-dateinputs@dev.

Both comments and separate issues are fine. I hope most people subscribed to this thread will find the early warnings useful.

@servelec-jack-starkey
Copy link

servelec-jack-starkey commented Nov 26, 2019

Hi, I'm having similar issues while trying to build in ivy mode with a grid:

ERROR in ../../node_modules/@progress/kendo-angular-grid/dist/es2015/index.d.ts:101:9 - error TS2304: Cannot find name 'PDFMarginComponent'.
101 export {PDFMarginComponent as PDFMarginComponent$1};
            ~~~~~~~~~~~~~~~~~~
../../node_modules/@progress/kendo-angular-grid/dist/es2015/index.d.ts:102:9 - error TS2304: Cannot find name 'PDFTemplateDirective'.
102 export {PDFTemplateDirective as PDFTemplateDirective$1};
            ~~~~~~~~~~~~~~~~~~~~
../../node_modules/@progress/kendo-angular-grid/dist/es2015/pdf/pdf.module.d.ts:38:25 - error TS7016: Could not find a declaration file for module '../../fesm2015/index'. '/home/[repo]/node_modules/@progress/kendo-angular-grid/dist/fesm2015/index.js' implicitly has an 'any' type.
38 import * as ɵngcc2 from '../../fesm2015/index';
ERROR in node_modules/@progress/kendo-angular-grid/dist/es2015/index.d.ts(101,9): error TS2304: Cannot find name 'PDFMarginComponent'.
    node_modules/@progress/kendo-angular-grid/dist/es2015/index.d.ts(102,9): error TS2304: Cannot find name 'PDFTemplateDirective'.
    node_modules/@progress/kendo-angular-grid/dist/es2015/pdf/pdf.module.d.ts(38,25): error TS7016: Could not find a declaration file for module '../../fesm2015/index'. '/home/[repo]/node_modules/@progress/kendo-angular-grid/dist/fesm2015/index.js' implicitly has an 'any' type.

Anything you can advise that I do to get it to compile? Thanks!

@tomvanoost
Copy link

@servelec-jack-starkey Same situation here. I get these errors on the latest development build.

@tsvetomir
Copy link
Member

We don't have a workaround for that particular issue yet. It looks like a backwards-compatibility issue with ngcc thay we hope to get solved in the upcoming Angular RCs.

@servelec-jack-starkey
Copy link

Hi @tsvetomir do you have a link to the issue in the Angular repo?

@tsvetomir
Copy link
Member

It's not in the public issue tracker AFAIK, see the ngcc failures summary document.

@servelec-jack-starkey
Copy link

It's not in the public issue tracker AFAIK, see the ngcc failures summary document.

Thanks @tsvetomir, I may be reading that incorrectly but the colour coding indicates it is a library (Kendo) issue?

@tsvetomir
Copy link
Member

I don't have enough information at the moment, but it may very well be an issue with our code.

@gradx
Copy link

gradx commented Dec 11, 2019

This is an issue for Angular 9.0.0-rc.5 as well

@tsvetomir
Copy link
Member

Compilation issues are fixed in Angular 9.0.0-rc.7 through angular/angular#34254.

@tsvetomir
Copy link
Member

Closing this issue as the compilation errors are now solved for all components. That is not to say that the components are fully compatible with Ivy just yet—we'll validate this in the upcoming days.

@bwinchester
Copy link

Just upgraded to 9.0.0-rc7-next to get this to work, seems to still be broken...reallly dont like paying for this.
image

@tsvetomir
Copy link
Member

@bwinchester can you please file a new issue with more details about the error you're getting? Thanks!

@0-gravity
Copy link
Author

Project from scratch gets compiled with RC7.

@0-gravity
Copy link
Author

Just upgraded to 9.0.0-rc7-next to get this to work, seems to still be broken...reallly dont like paying for this.
image

Have you tried to upgrade components version?

@bwinchester
Copy link

Issue was Ivy was still set to true for compile... all my fault.

Just upgraded to 9.0.0-rc7-next to get this to work, seems to still be broken...reallly dont like paying for this.
image

Have you tried to upgrade components version?

@MaklaCof
Copy link

MaklaCof commented Jan 7, 2020

@tsvetomir In this comment, you said:

Closing this issue as the compilation errors are now solved for all components. That is not to say that the components are fully compatible with Ivy just yet—we'll validate this in the upcoming days.

Is there anything new about official support for Angular 9 for all components?

@tsvetomir
Copy link
Member

The components are compatible with the latest release candidates with some minor exceptions. We're currently tracking a few issues with the Sortable component that we hope to fix soon.

@tscislo
Copy link

tscislo commented Feb 13, 2020

@tsvetomir does that mean that I need to bump all kendo components to latest version and it should be fine to compile with Ivy and Angular 9.0.0?

@tsvetomir
Copy link
Member

@tscislo yes, all latest versions of the packages are compatible with Angular 9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests