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

issue with angular core dependency #301

Closed
summerbasehart opened this issue Apr 1, 2022 · 5 comments
Closed

issue with angular core dependency #301

summerbasehart opened this issue Apr 1, 2022 · 5 comments

Comments

@summerbasehart
Copy link

Hello,

Trying to do a build in angular and I am getting some core dependency errors

Error: ./node_modules/ngx-logger/fesm2015/ngx-logger.mjs 900:58-80
Can't import the named export 'ɵɵngDeclareInjector' from non EcmaScript module (only default export is available)

Error: ./node_modules/ngx-logger/fesm2015/ngx-logger.mjs 899:58-80
Can't import the named export 'ɵɵngDeclareNgModule' from non EcmaScript module (only default export is available)

I tried updating core and angular cli, but I still have the errors

@minademian
Copy link

Had the same error. I'm running Angular 11. Documentation says that the library supports 6+.

@danielteles
Copy link

danielteles commented May 24, 2022

I was having the same problem here. Running Angular 11.2.3 and ngx-logger 5.0.9, so I installed the version 4.3.3 and the problem was solved.

P.S.: I chose the version 4.3.3 based on the number of the downloads on the npm page

@bmtheo
Copy link
Collaborator

bmtheo commented May 24, 2022

Thanks for sharing your solution

Documentation might not be up to date about versions that are supported

@bmtheo
Copy link
Collaborator

bmtheo commented May 24, 2022

Documentation (on github, npmjs will be updated when next publish happens) is updated with a warning about this topic, ideally we should have a "matching table" for versions

@bmtheo
Copy link
Collaborator

bmtheo commented Mar 20, 2023

Closing this issue, we are now using #308 to list all dependencies, once we have the full list it will be included in the doc

@bmtheo bmtheo closed this as completed Mar 20, 2023
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

No branches or pull requests

4 participants