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

An Error related to dayjs Upgrade #5087

Closed
mcdmaster opened this issue Jul 23, 2020 · 6 comments
Closed

An Error related to dayjs Upgrade #5087

mcdmaster opened this issue Jul 23, 2020 · 6 comments
Labels
bug バグ。本来あるべき動作をしていないもの

Comments

@mcdmaster
Copy link
Contributor

mcdmaster commented Jul 23, 2020

[訂正あり:当初 nuxt-i18n 関連と勘違いしていました。dayjs がらみですね]

起こっている問題 / The Problem

  • ビルド時に、ファイル plugins/chartjs-adapter-dayjs.ts が TS エラーを吐く

スクリーンショット / Screenshot

コメント 2020-07-23 095328

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

  • エラーの解消

起こっている問題の再現手段 / Steps to Reproduce

  • yarn dev (または yarn dev-no-axe)の実行
  • コンソールメッセージを確認

動作環境・ブラウザ / Environment

  • macOS / Windows / Linux / iOS / Android
  • Chrome / Safari / Firefox / Edge / Internet Explorer
@mcdmaster mcdmaster added the bug バグ。本来あるべき動作をしていないもの label Jul 23, 2020
@mcdmaster mcdmaster changed the title An Error related to nuxt-i18n Upgrade An Error related to dayjs Upgrade Jul 23, 2020
@kaizumaki
Copy link
Collaborator

@mcdmaster ご指摘ありがとうございます!私のローカル環境で yarn dev をやってみたところ、こちらのエラーは再現しないのですが、どうなんでしょう...?

@goki90210
Copy link
Contributor

@mcdmaster 私の環境でも出ません。#4918 の作業の関連でnode_modulesやyarn.lockが違っていたりしませんか?

@goki90210
Copy link
Contributor

@mcdmaster 今のバージョンからdayjsをアップグレードして、dayjs@1.8.30に上げたら出ましたね。

@goki90210
Copy link
Contributor

@mcdmaster
ここで報告されていたので、dayjsを1.8.29から1.8.30に上げると発生する問題で確定です。
iamkun/dayjs#964
上記のエラーは型をstringからロケールのtypeに変更したため発生した問題のようで、dayjs@1.8.31でロールバックされる予定です。
したがって、dayjs@1.8.30には上げずに、dayjs@1.8.31のリリースを待って対応しましょう。
(ソースの修正は不要になるっぽいです。)

@mcdmaster
Copy link
Contributor Author

りょうかいです。返答が遅くなってしまい失礼しました

@mcdmaster
Copy link
Contributor Author

dayjs@1.8.31 のリリースに伴い、問題の解決が見られたと判断しました。
よって本イシューはクローズします。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug バグ。本来あるべき動作をしていないもの
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants