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

Update to Angular v15 #64

Closed
wants to merge 2 commits into from
Closed

Update to Angular v15 #64

wants to merge 2 commits into from

Conversation

ectuser
Copy link
Contributor

@ectuser ectuser commented Dec 28, 2022

Type of change

  • Docs
  • Bug fix
  • Feature

Packages

  • @livechat/widget-core
  • @livechat/widget-react
  • @livechat/widget-vue
  • @livechat/widget-angular

Issue

Update to Angular v15.

Description

Previous Pull Request introduced update to Angular v14. This one updates the package and the example app to v15. Also updated jest, jest-preset-angular. Introduced config changes for jest.

Jest update was needed because of typescript update to v4.8 which is required by Angular v15

@ectuser ectuser requested a review from a team as a code owner December 28, 2022 09:59
This was referenced Dec 28, 2022
@ectuser
Copy link
Contributor Author

ectuser commented Dec 28, 2022

One thing I faced is the first run of tests performance that takes about 30s in my local machine. Please let me know if you face the same issue.

@ectuser
Copy link
Contributor Author

ectuser commented Jan 1, 2023

Hello @walaszczykm! Can you please take a look at the PR please. Thank you!

@walaszczykm
Copy link
Collaborator

Hi @ectuser, thanks for the PR. I will take a look at that and let you know 👍

@walaszczykm
Copy link
Collaborator

@ectuser changes look good, and tests pass ✅ I have checked the updated example App, and it works great 👍 I am going to verify the backward compatibility of the updated Angular package with previously supported versions to make sure we will not introduce any breaking changes with this update 🙂

@walaszczykm
Copy link
Collaborator

Hello @ectuser. I have run regression tests, and unfortunately, it looks like the upgraded Angular version used for building the package causes breaking compatibility changes 😢

I have run through projects with Angular: 12, 13, 14, and 15, and it looks like upgrading the package Angular version 15 causes incompatibility with Angular 12 and 13.

On the other hand, I have run the same regression tests with the @livechat/widget-angualr package built using Angular 13 a surprise it worked well without any issues on projects with Angular 12, 13, 14, 15. The only required thing was to extend the supported peer dependency on Angular, but despite that, nothing more was required.

Considering those results IMO, the Angular version for the package needs to be downgraded to Angular 13, and only peerDependecies inside its package.json should be updated to mark support for Angular 15.

We can move forward with the Angular version upgrade for the package when we are ready to drop support for Angular 12 and 13, but we want to do something other than that.

That means I would close this PR and prepare an appropriate downgrade on myself while keeping Angular 14 and 15 s supported peer dependency.

@walaszczykm walaszczykm closed this Jan 6, 2023
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