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

Conversation

mcdmaster
Copy link
Contributor

👏 解決する issue / Resolved Issues

📝 関連する issue / Related Issues

⛏ 変更内容 / Details of Changes

  • Revisit to the use of dayjs.local(), whose setter method changed from v1.8.30 to the newest

📸 スクリーンショット / Screenshots

  • No change is intended on any UI/UX.

@goki90210
Copy link
Contributor

goki90210 commented Jul 27, 2020

#5088 でも書きましたし、ここでもリンクされていますが、
iamkun/dayjs#964
[email protected]で行われた修正が[email protected]でロールバックされる予定なので、この対応は不要になると思われます。

@mcdmaster
Copy link
Contributor Author

@goki90210 ご確認をいただきありがとうございます。
dayjs 1.8.31 で問題が出なければ、イシュー・PR ともクローズします

@mcdmaster
Copy link
Contributor Author

本件、[email protected] が GA になりましたので、クローズします。
ローカル環境でも、特に問題は見られませんでした

@mcdmaster mcdmaster closed this Jul 29, 2020
@mcdmaster mcdmaster deleted the hotfix/issue-5087-chartjs-adapter-dayjs branch July 29, 2020 15:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

An Error related to dayjs Upgrade
2 participants