Skip to content

Conversation

@nZeus
Copy link
Contributor

@nZeus nZeus commented Jan 6, 2026

This PR updates the immer dependency so that this library can also be used with angular 21

@nZeus
Copy link
Contributor Author

nZeus commented Jan 8, 2026

cc: @WalterNorthwoods
Could you please review the PR?

@WalterNorthwoods
Copy link
Contributor

WalterNorthwoods commented Jan 8, 2026 via email

@nZeus
Copy link
Contributor Author

nZeus commented Jan 8, 2026

The current version is incompatible with the angular 21

npm error code ERESOLVE
npm error ERESOLVE could not resolve
npm error
npm error While resolving: [email protected]
npm error Found: [email protected]
npm error node_modules/immer
npm error   immer@"11.1.3" from the root project
npm error
npm error Could not resolve dependency:
npm error peer immer@"^10.1.1" from [email protected]
npm error node_modules/gojs-angular
npm error   gojs-angular@"2.1.3" from the root project
npm error
npm error Conflicting peer dependency: [email protected]
npm error node_modules/immer
npm error   peer immer@"^10.1.1" from [email protected]
npm error   node_modules/gojs-angular
npm error     gojs-angular@"2.1.3" from the root project

Ideally it's good to update the version of angular to v21, but that gives some weird errors:
image

So the safest option is to only update immer.

I did. Alas, when rebuilding I got a bunch of errors.

Have you tried to clean the folders first?

rm -Rf node_modules && rm package-lock.json
npm cache clean --force

@WalterNorthwoods
Copy link
Contributor

WalterNorthwoods commented Jan 8, 2026 via email

@simonsarris
Copy link
Contributor

We'll look at this today

@simonsarris simonsarris merged commit b2ed46e into NorthwoodsSoftware:master Jan 8, 2026
@nZeus nZeus deleted the immer-11.1.3 branch January 8, 2026 15:04
@simonsarris
Copy link
Contributor

Thanks for your patience Ilia. We just published [email protected], if you want to give it a try. I also removed zones as a dependency, which hopefully causes no issues.

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.

3 participants