Covalent: Add Angular 8 compatibility

Created on 29 May 2019  路  10Comments  路  Source: Teradata/covalent

Feature Request

Please make @covalent/core compatible with Angular 8

What is the expected behavior?

Covalent package works fine in Angular 8 project.

What is the motivation / use case for changing the behavior?

Angular 8 features

Which version of Angular and Material, and which browser and OS does this issue affect?

Angular 8

Other information

This error happening after angular 7.2 -> angular 8.0 migration. ("@covalent/core": "^2.0.1" is used)
image

Most helpful comment

Hi @emoralesb05 , any comment on this one ? Lot of us out here eager for an upgrade to angular@8 , and this seems to be the only blocker . @dlq84 's pending PR should give us a quick fix

All 10 comments

the export DOCUMENT was removed in this commit after being deprecated for a while: https://github.com/angular/angular/commit/3a9d2473ca1b5a5ef7595bcb58fb52b1677be76f#diff-247c56c4fdd0f71feca2f9373ba0980a

It now exists in @angular/common instead: https://github.com/angular/angular/commit/2a7ebbe982624c7cc617670f6f0284780de2d55d#diff-4410d13f4f850d652fed4180f23f7618

I have a branch that should fix this issue, want me to do a PR? https://github.com/dlq84/covalent/commit/d80f90178d2007722e988229ef5492e8005af21d

@dlq84 yes, of course)

Wow. That's a real deal breaker. I wonder why this wasn't fixed many versions ago. It's deprecated since two years.

Wow. That's a real deal breaker. I wonder why this wasn't fixed many versions ago. It's deprecated since two years.

Let's hope it doesn't take another two years to merge my PR ;)

Hi @emoralesb05 , any comment on this one ? Lot of us out here eager for an upgrade to angular@8 , and this seems to be the only blocker . @dlq84 's pending PR should give us a quick fix

I would also really like to see Angular 8 compatibility!

Any update on this when will update be available for angular 8 compatibility?

We are looking forward to upgrading to Angular 8. Would be great if this was tested and merged ASAP to ensure we can all upgrade our apps. Thank you!

We're also waiting for the Angular 8 compatible update. This is this only thing blocking us from upgrading to Angular 8.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

georgeedwards picture georgeedwards  路  4Comments

victorboissiere picture victorboissiere  路  5Comments

emoralesb05 picture emoralesb05  路  4Comments

mailok picture mailok  路  3Comments

d3viant0ne picture d3viant0ne  路  3Comments