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

chart.jsを2.9.4から4.4.1に更新 #1148

Merged
merged 3 commits into from Dec 31, 2023

Conversation

Na0ki
Copy link
Contributor

@Na0ki Na0ki commented Dec 30, 2023

表題の通りchart.jsをv2.9.4からv4.4.1に更新しました。
更新前後で挙動の違いはないようにしたつもりです。

更新に伴い下記対応を実施しました

  • chart.jsの更新
    • v3の設定の構造変更に追従
    • v4からのモジュールバンドル最適化(初期化時に必要なモジュールのみregister)に対応
  • @types/chart.jsの更新
  • webpack
    • color-name, chartjs-color, color-name, moment がchart.jsの依存から消えたため削除
    • @kurkle/color がchart.jsの依存に追加されたため追加
    • moment の externals の設定を削除

なお、本PRが取り込まれたらdependabotによる下記PRは閉じてよさそうです。
#1126

参考資料

更新に伴い下記対応を実施した
* chart.jsの更新
  * v3の設定の構造変更に追従
* @types/chart.jsの更新
* Chart初期化時に必要なモジュールのみをロードするようにした(v4系からの機能)
* webpack
  * color-name, chartjs-color, color-name, moment がchart.jsの依存から消えたため削除
  * @kurkle/color がchart.jsの依存に追加されたため追加
  * moment の externals の設定を削除
Copy link

codecov bot commented Dec 30, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (aaac28c) 41.55% compared to head (ecd9c52) 41.55%.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1148   +/-   ##
==========================================
  Coverage      41.55%   41.55%           
  Complexity       980      980           
==========================================
  Files            121      121           
  Lines           3598     3598           
==========================================
  Hits            1495     1495           
  Misses          2103     2103           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shibafu528 shibafu528 self-requested a review December 31, 2023 02:10
@shibafu528
Copy link
Member

Build container image / build (pull_request) が落ちていますがこれは無視してください。私のほうのミスです 🙏

Copy link
Member

@shibafu528 shibafu528 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

調査・対応いただき、ありがとうございます! 手が回っていなかったので助かります。

createLineGraph() で作成されるグラフの表示が若干異なり、塗り潰しがなくなっているのが気になりました。これはデフォルトオプションの仕様が変わったのでしょうか。

難しくなければ見た目を維持したいと思っていますが、可能でしょうか?

before after
image image

package.json Outdated Show resolved Hide resolved
@Na0ki
Copy link
Contributor Author

Na0ki commented Dec 31, 2023

@shibafu528 レビューありがとうございます。
年間グラフ背景については見落としの対応漏れでした。
fill 設定の追加および filler プラグインの追加で対応しました。
また、別途コメントした通り @types/chart.js について削除しました。

Copy link
Member

@shibafu528 shibafu528 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🙌

@shibafu528 shibafu528 merged commit a325f71 into shikorism:develop Dec 31, 2023
4 checks passed
@shibafu528 shibafu528 mentioned this pull request Jan 6, 2024
4 tasks
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.

None yet

2 participants