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

Angular 14. Fixes #153 #154

Closed
wants to merge 10 commits into from
Closed

Angular 14. Fixes #153 #154

wants to merge 10 commits into from

Conversation

jerone
Copy link

@jerone jerone commented Oct 24, 2022

This PR upgrades the whole project to support Angular 14.

Fixes #153 & #137 & #140 and probably many others!

  • Upgrade to Angular 14.
  • Allowed Angular 14 in dependencies range. Loosened other dependencies too (@types/fusioncharts, RxJS, Zone.js). Not good to pin those so hard.
  • Converted (unsupported) angular-cli.json to angular.json.
  • Fixed issues with rollup config and bulding.
  • Updated version to 14 to keep in sync with Angular versioning.
  • Updated dist folder. Why does this needs to be committed?

I got these change working in our companies project, but this requires full testing on all supported Angular versions, and probably even more clean up.
Why is there no automated testing setup?

Also, please don't publish the whole project to NPM, only the dist files are enough: https://unpkg.com/browse/[email protected]/

@jerone jerone changed the title Angular 14 Angular 14. Fixes #153 Oct 24, 2022
@paksydavid
Copy link

Many thanks. 👍🏽

Would be great if this PR would receive some maintainer attention.

@jerone
Copy link
Author

jerone commented Jan 20, 2023

Apparently FusionCharts is doing their own upgrade, which is good news (finally). Aldo some communication would be nice.

It also looks like they focus on Angular 15, which is already out for 2 months.

For that reason and with all these merge conflicts, I'm going to close this PR.

I do hope their upgrade is better then their review process; 1e29b5c#r97467805 and a few other weird changes.
I hope to be positively surprised.

@jerone jerone closed this Jan 20, 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.

Does not work with angular 14
2 participants