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

Fix build errors for angular 13 #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

khumbal
Copy link

@khumbal khumbal commented Apr 18, 2022

ModuleWithProviders should also be imported from '@angular/core' and not '@angular/compiler/src/core'

ModuleWithProviders should also be imported from '@angular/core' and not '@angular/compiler/src/core'
@touqir-asghar
Copy link

touqir-asghar commented Apr 23, 2022

admin please merge this PR to solve mentioned issue - as this package is unusable without this fix

@Sumragen
Copy link

Sumragen commented Jun 1, 2022

@gabrielbusarello Could you merge it?

@edwinsp94
Copy link

edwinsp94 commented Jun 12, 2022

I have the same problem for Angular 14.

Angular CLI: 14.0.1
Node: 16.15.1
Package Manager: npm 8.12.1
OS: win32 x64

Angular: 14.0.1
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, localize, material, material-moment-adapter
... platform-browser, platform-browser-dynamic, router

Package Version

@angular-devkit/architect 0.1400.1
@angular-devkit/build-angular 14.0.1
@angular-devkit/core 14.0.1
@angular-devkit/schematics 14.0.1
@angular/fire 7.4.1
@schematics/angular 14.0.1
rxjs 7.5.5
typescript 4.7.3

@cattingcat
Copy link

@gabrielbusarello It would be really great to merge it (and also add support for ng14)

@mm-pbr
Copy link

mm-pbr commented Sep 8, 2022

@gabrielbusarello could you please merge it for Angular 13

@BelvedereHenrique
Copy link

@gabrielbusarello duvido fazer o merge...

@anilarya-spacer
Copy link

@gabrielbusarello : Please merge it .

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

Successfully merging this pull request may close these issues.

9 participants