search

traPtitech / traQ_S-UI

traQ S - traP Internal Messenger Application Frontend
MIT License
109 stars 40 forks source link

traQ上のメッセージの投稿・編集日時に年月日を適宜省略しつつ付け足す #4338

Closed kitsne241 closed 4 months ago

kitsne241 commented 5 months ago

https://github.com/traPtitech/traQ_S-UI/issues/3746

↑に関連していますが、当初のIssueが求める通りの実装ではありません。

表示するよう date.ts内の関数getDisplayDate を書き換えました。 開発環境ではうまくいっているように見えましたが、この変更がtraQに表示されるメッセージの日付以外に影響を与える可能性について把握し切れていないのでレビューをお願いしたいです。

スクリーンショット 2024-07-01 15 56 59
github-actions[bot] commented 5 months ago

Preview (prod) → https://4338-prod.traq-preview.trapti.tech/ Preview (dev) → https://4338-dev.traq-preview.trapti.tech/

kitsne241 commented 4 months ago

変更自体は問題ないと思います! 古いメッセージがいつ投稿されたものなのかわかりやすくなって嬉しいです 現状、testが落ちてしまっているので、その修正と、「今日の場合」「昨日の場合」「今年の場合」のtestも追加してほしいです! testの修正・追加でわからないことがあったら気軽にtraQなどで質問してください!

ありがとうございます！testを修正しました。

具体的には、現在日時と編集日時との関係によって日付表示が『YYYY/MM/DD』『MM/DD』『昨日』『今日』の4通りに変化するので、それぞれについて正しく日付表示が場合分けされているかを検証するように書き換えました。現在日時を考慮してテストケースを作成する必要があるので多少コードが煩雑になってしまいました（とくに『MM/DD』の表示条件に合致する日付を常に生成するために工夫が必要でした）

kitsne241 commented 4 months ago

なるほど、現在時刻を動かせるんですね…。勉強になります。確かにシンプルに書けました