Closed kuniyuki-f closed 1 year ago
https://63d52217f1430a5ad69846cd-hveofpourd.chromatic.com/?path=/story/templates-timertemplate--default
基本的な実装の流れは #93, #96 に類似しています。 上記 PR の実装の流れを踏襲していますので、そこまで大きな論点はない認識です。
Note 本 PR は、 #93 , #96 で実装した、「計測中のタスク一覧を表示する」の停止中バージョンです。 APIリクエスト機能と表示機能をまとめてPR出しているため、コミット行が多くなってしまい申し訳ありません🙇
1点、補足情報に今後の展望を記載していますので、ご確認お願いします! 気になる点があればレビューコメントいただけると幸いです🙏
このPRがマージされたら、次は以下のタスクを実施する予定です。
上記の機能を MeasuringTaskItem, StoppedTaskItemコンポーネントにそれぞれ組み込み、 #81 で議論になっていた「2つのコンポーネントを1つのTaskItemコンポーネントに統合する」を実施するつもりです。 他に優先度高そうなタスクありましたら、ご提案ください!
MeasuringTaskItem
StoppedTaskItem
TaskItem
以上、よろしくお願いします!
The latest updates on your projects. Learn more about Vercel for Git ↗︎
issueURL
60
この PR で対応する範囲 / この PR で対応しない範囲
Storybook の URL、 スクリーンショット
https://63d52217f1430a5ad69846cd-hveofpourd.chromatic.com/?path=/story/templates-timertemplate--default
変更点概要
レビュアーに重点的にチェックして欲しい点
基本的な実装の流れは #93, #96 に類似しています。 上記 PR の実装の流れを踏襲していますので、そこまで大きな論点はない認識です。
1点、補足情報に今後の展望を記載していますので、ご確認お願いします! 気になる点があればレビューコメントいただけると幸いです🙏
補足情報
このPRがマージされたら、次は以下のタスクを実施する予定です。
53
57
上記の機能を
MeasuringTaskItem
,StoppedTaskItem
コンポーネントにそれぞれ組み込み、 #81 で議論になっていた「2つのコンポーネントを1つのTaskItem
コンポーネントに統合する」を実施するつもりです。 他に優先度高そうなタスクありましたら、ご提案ください!以上、よろしくお願いします!