flauc / angular2-notifications

A light and easy to use notifications library for Angular.
https://stackblitz.com/edit/angular2-notifications-example
MIT License
746 stars 163 forks source link

issue with lazy loaded module in angular 13 #392

Closed cavone-leonardo closed 2 years ago

cavone-leonardo commented 2 years ago

Use Case

I reproduced the problem in this example:

Description

Using angular2-notifications version 12 in a lazy loaded module, you get the following error:

ERROR Error: Uncaught (in promise): Error: BrowserModule has already been loaded. If you need access to common directives such as NgIf and NgFor from a lazy loaded module, import CommonModule instead.

Angular v. 13.2.5

image

Nomis87 commented 2 years ago

Same for me with Angular 12

wwjhu commented 2 years ago

Any chance you can create a 12.0.1 release with this fix any time soon?

maychan111 commented 1 year ago

Same issue here.

@flauc can we have a 12.0.1 release please?

@wwjhu how are you getting around this issue with no 12.0.1?

wwjhu commented 1 year ago

@wwjhu how are you getting around this issue with no 12.0.1?

I created a private fork that I hope to get rid of once a release is here.