Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Upgrading chart.js up to @ 3.x #6163

Open
mcdmaster opened this issue Apr 3, 2021 · 2 comments
Open

Upgrading chart.js up to @ 3.x #6163

mcdmaster opened this issue Apr 3, 2021 · 2 comments
Labels
help-wanted 特に助けを必要としているもの improvement 改善や新機能の要望

Comments

@mcdmaster
Copy link
Contributor

改善詳細 / Details of Improvement

スクリーンショット / Screenshot

N/A

期待する見せ方・挙動 / Expected behavior

  • chart.js 2.x -> 3.x のアップグレードは、vue 2 -> 3 へのアップグレードと同程度のドラスティックな変更が求められると考えます ※参考 [Draft] Feat/vue3+webpack5 #5667
  • まずは変更要件を整理したうえで、具体的に何を変更すべきか、対象アプリを一気に変更するか(ビッグバンアプローチ)、もしくは段階的に行うか(ウェーブ・アプローチ)、ウェーブ・アプローチをとる場合に優先順位を決める基準は何か、といったあたりを明確化することが求められます
  • 上記が決まれば、あとは実行あるのみです

動作環境・ブラウザ / Environment

  • macOS / Windows / Linux / iOS / Android
  • Chrome / Safari / Firefox / Edge / Internet Explorer
@mcdmaster mcdmaster added the improvement 改善や新機能の要望 label Apr 3, 2021
@kaizumaki
Copy link
Collaborator

@mcdmaster おお、フォローありがとうございます!chart.js v3のmigrationのドキュメント見ましたが、結構細々とありますね。まだちゃんと把握してないのですけど、一旦整理したいですね。もしかしたら、一気に変更も可能かもしれません。

@kaizumaki kaizumaki added the help-wanted 特に助けを必要としているもの label Apr 5, 2021
@mcdmaster
Copy link
Contributor Author

@kaizumaki
これ↓を読むと、けっこう多岐に及ぶことに気づきます。
https://www.chartjs.org/docs/latest/getting-started/v3-migration.html

どうやらオプションやプロパティの変数名まで変わってしまうようなので、安易な見立てをするなら、チャート系のアプリは全修正対象ですね。
ただ、plugins/vue-chart.ts の書き方か何かで逃げる余地はあるかもしれません。あるいはそれ用のモジュールが出ているとありがたいのですけれども…

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help-wanted 特に助けを必要としているもの improvement 改善や新機能の要望
Projects
None yet
2 participants