Closed kuniyuki-f closed 1 year ago
Storybook
とくになし
補足情報に記載の通り、既存のコンポーネントのほぼコピーです。 したがって、そこまでレビューに時間をかけていただく必要はない認識です。
こちらのやり取りから、一旦計測中のタスクと停止中のタスクのコンポーネントを分けて実装し、後でTaskItemなどの名前で1つのコンポーネントに統合する予定です。
TaskItem
したがって、本PRで実装したコンポーネントは、 #82 と配色が異なるだけで現状ほぼ同等のコンポーネントになっています。 ※ API ロジック実装時に処理内容やPropsなどの差異が出てくる想定です。 その差異を吸収できるなら統合、できないならそのまま別々のコンポーネントにするのもありという認識です。
The latest updates on your projects. Learn more about Vercel for Git ↗︎
issueURL
81
この PR で対応する範囲 / この PR で対応しない範囲
Storybook の URL、 スクリーンショット
Storybook
変更点概要
とくになし
レビュアーに重点的にチェックして欲しい点
補足情報に記載の通り、既存のコンポーネントのほぼコピーです。 したがって、そこまでレビューに時間をかけていただく必要はない認識です。
補足情報
こちらのやり取りから、一旦計測中のタスクと停止中のタスクのコンポーネントを分けて実装し、後で
TaskItem
などの名前で1つのコンポーネントに統合する予定です。したがって、本PRで実装したコンポーネントは、 #82 と配色が異なるだけで現状ほぼ同等のコンポーネントになっています。 ※ API ロジック実装時に処理内容やPropsなどの差異が出てくる想定です。 その差異を吸収できるなら統合、できないならそのまま別々のコンポーネントにするのもありという認識です。