-
Notifications
You must be signed in to change notification settings - Fork 120
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
Angular 14 deepmerge warning #229
Comments
Me too see this warnings and we want and update |
Same here. |
Similar problem with angular 14.2.0
|
Hi @BBlackwo, |
There are two options to fix this:
|
@BBlackwo we'll see there is talk about them adding esm, i'd love to see this move forward, this is kind of out of date with standards there are already 2 libs that added support that are named ts-deepmerge and deepmerge-ts lol |
https://snyk.io/advisor/npm-package/ts-deepmerge has is healthier than the original, and the package size is reduced seems like a really good choice imo. Also this issue was reported back in angular 14 (we're now on angular 16) also this fix seems pretty prescribed and simple? |
Migrated the deepmerge library to ts-deepmerge for better TypeScript compatibility and enhanced type checking. fixes btroncone#229
Migrated the deepmerge library to ts-deepmerge for better TypeScript compatibility and enhanced type checking. fixes btroncone#229
Migrated the deepmerge library to ts-deepmerge for better TypeScript compatibility and enhanced type checking. fixes btroncone#229
Migrated the deepmerge library to ts-deepmerge for better TypeScript compatibility and enhanced type checking. fixes btroncone#229
Migrated the deepmerge library to ts-deepmerge for better TypeScript compatibility and enhanced type checking. fixes btroncone#229
Migrated the deepmerge library to @bundled-es-modules/deepmerge. fixes btroncone#229
You can use |
Could you provide an example? |
Versions
ngrx-store-localstorage 14.0.0
Angular 14.0.5
The text was updated successfully, but these errors were encountered: