-
Notifications
You must be signed in to change notification settings - Fork 51
Core - update : angular v16 #100
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
Conversation
Nice one! pls merge it if you have time =) @AlexMiniApps |
FYI guys this branch is also working well for me on Angular 16 w/ Ionic 7 |
Is this repo maintained? Is there a published fork with Angular 16 support? |
@Ice32 Yes it is and I had personal contact with the maintainer, @AlexMiniApps. He's just a bit in a rush at the moment and couldn't find time to merge my request. That said, if you have only one project, you could download my fork locally , rename the You may then have to |
Hopefully this will be merged soon. Otherwise going to have to remove the package from project as it's blocking a release. |
Le me know when will it be merged. Because I stopped using this package in my project because this couldn't be used in Angular 16. |
We also would like to see this PR merged 👍 |
@AlexMiniApps Thanks for merging, when can we expect the new release containing the changes? |
@AlexMiniApps |
Description
I've done multiple commit to split every change I made, but I'll comment them here.
Done
v16.0.2
tsickle
library It wasn't used anymore and was quite oldnpm run publish
to make life easierconfiguration --production
to change the compilation mode -> package get reduced from207kb
down to176kb
Additional
I've tested it on my angular v16 project, works like a charm.
But a second control is never bad
Resolve
The following shall be closed, since everything is contained in this release