ngx-bootstrap-multiselect icon indicating copy to clipboard operation
ngx-bootstrap-multiselect copied to clipboard

[feat] Update to Angular 14

Open sommmen opened this issue 2 years ago • 4 comments

Hiya @softsimon,

The component works as-is on Angular 14, but ofcourse i'm now getting a bunch of errors:

PS C:\Repos\Front-End> npm install        
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving: [email protected]
npm ERR! Found: @angular/[email protected]
npm ERR! node_modules/@angular/common
npm ERR!   @angular/common@"^14.2.1" from the root project
npm ERR!   peer @angular/common@"^14.0.0 || ^15.0.0" from @angular/[email protected]
npm ERR!   node_modules/@angular/cdk
npm ERR!     @angular/cdk@"^14.2.1" from the root project
npm ERR!   9 more (@angular/forms, @angular/platform-browser, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer @angular/common@"^10.0.6" from [email protected]
npm ERR! node_modules/ngx-bootstrap-multiselect
npm ERR!   ngx-bootstrap-multiselect@"^2.1.1" from the root project
npm ERR!
npm ERR! Conflicting peer dependency: @angular/[email protected]
npm ERR! node_modules/@angular/common
npm ERR!   peer @angular/common@"^10.0.6" from [email protected]
npm ERR!   node_modules/ngx-bootstrap-multiselect
npm ERR!     ngx-bootstrap-multiselect@"^2.1.1" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\[REDACTED]\AppData\Local\npm-cache\eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\[REDACTED]\AppData\Local\npm-cache\_logs\2022-09-15T07_34_52_319Z-debug-0.log

This is forcing me to use --force when running npm install to make the project work. Could this please be updated to the latest Angular version?

I'll be happy to shoot in a PR - i'm not JS dev, but this seems like an easy task. Just let me know.

sommmen avatar Sep 15 '22 07:09 sommmen

I'll be happy to shoot in a PR

I would be happy to merge it ;)

softsimon avatar Sep 15 '22 11:09 softsimon

I'll be happy to shoot in a PR

I would be happy to merge it ;)

K - I hope someone who's an actual js dev could chip in but ill try and make a PR if i can find some time. Would it be a good idea to make packages for angular versions inbetween or should we move directly to v14?

@kek-Sec FYI perhaps you'd wanna help.

sommmen avatar Sep 15 '22 11:09 sommmen

I'd be down to give this a try , tbh this project's deps have stayed quite a bit behind

kek-Sec avatar Sep 15 '22 13:09 kek-Sec

I'd be down to give this a try , tbh this project's deps have stayed quite a bit behind

Go for it if you have the time 😄. I got holiday coming up so before i have any time it will be a while.

sommmen avatar Sep 15 '22 14:09 sommmen

I'd be down to give this a try , tbh this project's deps have stayed quite a bit behind

Bump. Do you reckon you have some time to update to v14? v15 is out aswell, but personally i'm stuck in v14 for a while due to other libraries.

sommmen avatar Nov 21 '22 10:11 sommmen

Any update on the status of upgrading to angular v14?

I'm trying to use it but I'm getting: NgxBootstrapMultiselectModule does not have a module def (ɵmod property)

bertdewolf avatar Jan 05 '23 13:01 bertdewolf

@softsimon I created the following PR: https://github.com/softsimon/ngx-bootstrap-multiselect/pull/485 This should make the library compatible with Angular 15. I hope you have time to have a look at it.

dsteegen avatar Mar 05 '23 20:03 dsteegen

@softsimon Kind reminder to check my previous comment. If you do not maintain this library anymore, we will have to migrate to a fork of your library (https://github.com/fast-facts/ngx-bootstrap-multiselect) which is already compatible with the Ivy compiler (Angular 12 and beyond).

dsteegen avatar Mar 13 '23 12:03 dsteegen

@softsimon thank you merging my PR already :D I didn't expect that you would approve it so fast! May I assume that you will also release a new version to the public NPM repository anytime soon? ;)

dsteegen avatar Mar 13 '23 13:03 dsteegen

Latest v on npm saids;

"peerDependencies": { "@angular/common": ">=15.0.0", "@angular/core": ">=15.0.0" },

So this has long been done :). Updating now!

sommmen avatar Jun 15 '23 07:06 sommmen