-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add support for Angular 17 #62
Comments
Angular 17 has breaking change, that affect change detection (and ngxd):
|
any ETA on when Angular 17 will be supported? We're currently having issues described by @janousek and would love to have an official solution, rather than a hacky workaround :) |
Regarding the above PR. For those of you who'd like to get moving before this is merged and published, the main change is here: I had to make a change to some of the test code because Angular now handles OnPush and Default change notification differently. That change is here: https://github.com/Indigosoft/ngxd/pull/63/files#diff-cfe523e683be6cacaebf3ed3129dd8d981995a7e9e3222281bda90f3062aa1b0L10-R23 The other changes are just upgrading to Angular 17 |
Any ETA for the new release? |
Any ETA for the new release? @diogogasparr @DaveMBush @thekiba |
Wanted to know if there is any update on this? |
was this project dropped? |
Since Angular v16.2, this library is not needed anymore for probably the majority of the use cases:
|
@realbizkit, if you have a lot of legacy code that already uses this library, you still need it so you can make the transition. Anyone who is using pnpm or yarn can patch the library with my fix that is still awaiting approval. |
Describe the feature you would like to see added
Add support for Angular 17
Describe the solution you'd like
Update peer dependencies to allow angular 16:
Describe alternatives you have considered
The text was updated successfully, but these errors were encountered: