fusioncharts / angular-fusioncharts

Angular Component for FusionCharts JavaScript Charting Library
https://fusioncharts.github.io/angular-fusioncharts/#/ex1
Other
55 stars 37 forks source link

[feature] [SUPPORT-2357] upgrade angular version from v14 to v17. (#186) #187

Closed meherhowji-5740 closed 4 months ago

meherhowji-5740 commented 5 months ago

🚀 Updates angular from 14.x to 17.3.6 ✏️ used angular official documentation for upgrading the version (ng update)

Changelogs

Sourced from angular's changelog.

File Changes
  • angular.json
  • package.json
  • package-lock.json
  • tsconfig.json

🔨 Replaces ng2-codemirror from with ngx-codemirror ✏️ ng2-codemirror is no longer working in Angular-v16 and above and is not maintained from last 7 years.

ngx-codemirror v7 is open source with MIT license. It uses the same peer dependency codemirror, as ng2-codemirror does.

File Changes
  • package.json
  • package-lock.json
  • src/app/app.component.html
  • src/app/app.module.ts
  • src/app/fusioncharts/tabbedComponent/tabbed.code.html

🚀 Added a demo example application into examples/fc-angular-chart-demo

File Changes
  • everything in examples folder

🚀 Bumped Integration version from 4.0.3 to 4.1.0

File Changes
  • package.json
  • dist/package-dist.json
  • dist/package-lock.json
  • package-dist.json
  • package-lock.json
  • Readme.md

Other File Changes

Changed the type of input prop 'chartConfig' to same as 'dataSource' as it can also accept string and object.

  • src/angular-fusioncharts/src/fusioncharts.component.ts

Updated rollup config file(used in the build script), as the docs and config has changed.

  • rollup.config.js

Fixed compilation errors.

  • src/polyfills.ts
  • src/tsconfig.app.json

Made a new build using 'angular run build' command.

  • dist/dist/index.js
  • dist/dist/index.min.js
  • dist/src/fusioncharts.component.d.ts
  • dist/src/fusioncharts.component.js
  • dist/src/fusioncharts.component.js.map
  • dist/src/fusioncharts.directive.d.ts
  • dist/src/fusioncharts.service.js.map
  • yarn.lock
meherhowji-5740 commented 5 months ago

@kruzznikam if you can squash the commits then let's merge this to develop. everything else looks good to me.

meherhowji-5740 commented 5 months ago

@kruzznikam also, let's bump the integration version from 4.0.3 to 4.1.0