This repository has been archived by the owner on Jun 1, 2023. It is now read-only.
日付の表示形式を揃える #5077
Merged
日付の表示形式を揃える #5077
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shgtkshruch
commented
Jul 22, 2020
components/DataViewTable.vue
Outdated
Comment on lines
86
to
88
| formatDate(dateString) { | ||
| if (dayjs(dateString).isValid()) { | ||
| return this.$d(new Date(dateString), 'dateWithoutYear') |
There was a problem hiding this comment.
テーブルの日付はここで一括して M月D日 形式にしています。
Comment on lines
-289
to
+282
| { text: dayjs(label).format('M/D') }, | ||
| { text: label }, |
There was a problem hiding this comment.
各テーブルからは日付を指定するコードを削除しています。
utils/formatDayBeforeRatio.ts
Outdated
Comment on lines
31
to
35
| return [ | ||
| Vue.prototype.$nuxt.$options.i18n.d(new Date(lastDay), 'dateWithoutYear'), | ||
| formatter(lastDayData), | ||
| formatDayBeforeRatio(dayBeforeRatio, formatter) | ||
| ] |
There was a problem hiding this comment.
グラフ上部の前日比に関しては、displayData から
M月D日にフォーマットされた日付文字列- 最新データ
- 小数点を考慮して3桁区切りにした前日比
を返す関数を定義してロジックを共通化しました。
グラフごとに .DataView-DataInfo-date に入る文言が微妙に異なるので、Vue コンポーネントにはせずに計算結果のみを返す関数にしました。
各コンポーネントでは、この値を使って文言を組み立てる処理だけ残してあります。
munierujp
reviewed
Jul 22, 2020
utils/formatDayBeforeRatio.ts
Outdated
| displayData, | ||
| dataIndex = 0, | ||
| digit = 0 | ||
| }: DayBeforeRatioParameters): string[] { |
There was a problem hiding this comment.
戻り値の型をinterfaceで定義して、オブジェクトとして返したほうがよいかと思います。
There was a problem hiding this comment.
コメントありがとうございます！
戻り値の型定義あったほうが良かったですね...
こちらのコミットで修正しました。
aec731e
munierujp
reviewed
Jul 22, 2020
components/DataViewTable.vue
Outdated
| @@ -76,6 +81,15 @@ const options: ThisTypedComponentOptionsWithRecordProps< | |||
| dataKeys() { | |||
| return this.headers.map(h => h.value).filter(h => h !== this.headerKey) | |||
| } | |||
| }, | |||
| methods: { | |||
| formatDate(dateString) { | |||
Open
|
@shgtkshruch ご対応ありがとうございます！方向性としてはいい感じだと思います。 非表示になるカード
以下、長いですがコンソールに表示されたエラーを貼っておきますね（「その他参考指標」タブを選択したときの表示です）。 |
|
@kaizumaki
すでに導入している Day.js が IE 11 もサポートしているので、今回の影響範囲に関してはこちらで日付処理をしました。
|
|
@shgtkshruch ご対応ありがとうございます！IE11での正常動作を確認しました。 |
|
@shgtkshruch すみません！コンフリクトを起こしているようなので、解消しておいていただけると助かります 🙏 |
GitHub Actions log ``` [error][eslint] reported by reviewdog 🐶 `@/utils/formatDayBeforeRatio` import should occur before import of `~/utils/monitoringStatusValueFormatters` import/order Raw Output: 104:1 error `@/utils/formatDayBeforeRatio` import should occur before import of `~/utils/monitoringStatusValueFormatters` import/order [error][eslint] reported by reviewdog 🐶 `@/utils/formatDayBeforeRatio` import should occur before import of `~/utils/monitoringStatusValueFormatters` import/order Raw Output: 126:1 error `@/utils/formatDayBeforeRatio` import should occur before import of `~/utils/monitoringStatusValueFormatters` import/order ```
b10175f to
071d704
Compare
|
@kaizumaki |
|
@shgtkshruch ありがとうございます！ESLintの件はなるほどですね。対応してみます。 |
kaizumaki
approved these changes
Jul 28, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
👏 解決する issue / Resolved Issues
📝 関連する issue / Related Issues
⛏ 変更内容 / Details of Changes
data-view-tableコンポーネントで一括して指定するようにしましたformatDayBeforeRatioや前日比を計算するロジックを削除しましたTimeStackedBarChartのみデータ型が二次元配列で他のグラフを異なっているなっているので、これまでの実装を残しています📸 スクリーンショット / Screenshots
カード上部の日付のフォーマットについては、「その他 参考指標」タブを選択した後に、Chrome のコンソールから以下のコマンドでテキストを抽出して確認しました。