Skip to content
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

Updating to Ng 14 #78

Closed
wants to merge 14 commits into from
Closed

Updating to Ng 14 #78

wants to merge 14 commits into from

Conversation

chrux
Copy link
Contributor

@chrux chrux commented Jul 12, 2022

Merge or review #79 first

@chrux chrux changed the title Updating to Ionic 6 and Ng 14 WIP: Updating to Ionic 6 and Ng 14 Jul 12, 2022
@chrux
Copy link
Contributor Author

chrux commented Jul 12, 2022

I forgot to migrate to eslint, will do the change first

chrux added 3 commits July 12, 2022 11:59
Instead of tslint which is deprecated.
Instead of tslint which is deprecated.
@chrux chrux changed the title WIP: Updating to Ionic 6 and Ng 14 Updating to Ionic 6 and Ng 14 Jul 13, 2022
@chrux chrux changed the title Updating to Ionic 6 and Ng 14 WIP: Updating to Ionic 6 and Ng 14 Jul 13, 2022
@chrux chrux changed the title WIP: Updating to Ionic 6 and Ng 14 Updating to Ng 14 Jul 13, 2022
@chrux chrux closed this Jul 13, 2022
@chrux chrux deleted the i/ng-14 branch July 13, 2022 19:14
@arielfaur
Copy link
Owner

Hey @chrux ! It would be great if you could do the upgrade as I don't have time now. I see you closed your PRs. Any issues with that?
I can look at your PR over the weekend. Thanks!

@chrux
Copy link
Contributor Author

chrux commented Jul 21, 2022

Hello, thanks for replying so quickly and sorry I couldn't answer before, this the PR with the changes, pls let me know what you think, #81, also I had to revert Ng 14 as this plugins for cordova was causing some issues @ionic/cordova-builders, but they just released version 7 which fixes the issue, ionic-team/angular-toolkit#474 (comment). I will create another PR once I had time to double check is working on my project.

@arielfaur
Copy link
Owner

Hey @chrux
I merged your PR, great work thanks! Also, I updated the demo project 🎉

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.

2 participants