Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

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

Merged
AlexMiniApps merged 6 commits into AlexMiniApps:master from rbalet:core-angular-update
Jun 15, 2023
Merged

Core - update : angular v16 #100

AlexMiniApps merged 6 commits into AlexMiniApps:master from rbalet:core-angular-update
Jun 15, 2023

Conversation

@rbalet
Copy link
Contributor

@rbalet rbalet commented May 24, 2023 •
edited
Loading

Description

I've done multiple commit to split every change I made, but I'll comment them here.

Done

  1. Update up to angular v16.0.2
  2. Update every dependencies to their latest stable version
  3. Remove unused tsickle library It wasn't used anymore and was quite old
  4. Add a new script npm run publish to make life easier
  5. Add a new configuration --production to change the compilation mode -> package get reduced from 207kb down to 176kb
  6. Update the library from 1.6.0 to 1.7.0

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

zurek-piotr, axi92, and Wahsner reacted with thumbs up emoji drakedeatonuk and Ice32 reacted with eyes emoji
Copy link

axi92 commented May 30, 2023

Nice one! pls merge it if you have time =) @AlexMiniApps

drakedeatonuk reacted with eyes emoji

Copy link

FYI guys this branch is also working well for me on Angular 16 w/ Ionic 7
axi92 reacted with thumbs up emoji

Copy link

Ice32 commented Jun 6, 2023

Is this repo maintained? Is there a published fork with Angular 16 support?

Copy link
Contributor Author

rbalet commented Jun 6, 2023

@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 lib folder to angular-core-input and use it directly.

You may then have to npm i some dependencies
drakedeatonuk, Ice32, and 0x7061 reacted with heart emoji

Copy link

Wahsner commented Jun 6, 2023

Hopefully this will be merged soon. Otherwise going to have to remove the package from project as it's blocking a release.

Copy link

ngdevc commented Jun 15, 2023

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.

Copy link

We also would like to see this PR merged 👍

@AlexMiniApps AlexMiniApps merged commit f8ef761 into AlexMiniApps:master Jun 15, 2023
Copy link

@AlexMiniApps Thanks for merging, when can we expect the new release containing the changes?
Ice32, drakedeatonuk, and ngdevc reacted with thumbs up emoji axi92 reacted with hooray emoji

Copy link

ngdevc commented Jun 19, 2023

@AlexMiniApps
Thank you very much for merging.
drakedeatonuk reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /