Closed kuniyuki-f closed 1 year ago
vercel[bot]
commented
1 year ago The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| timelogger-web | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Aug 19, 2023 10:51am |
keitakn
commented
1 year ago @c501306014
https://github.com/commew/timelogger-web/commit/4d9628ccc807f14b721a1a357cace2c035345887 のコミットについて、chromatic のビルドがこけているようなので原因を調査する必要があります
ですがGitHubActions上の https://github.com/commew/timelogger-web/actions/runs/5910978638/job/16034671201?pr=106 から再度実行したら成功しましたね、あまり原因などは分かっていないのですが、一度ビルドが失敗するともう一回再トライしないとその後はずっと失敗し続けてしまうという事になっている気がします💦（何でそうなっているのかは分かりません）
この件はちょっと調べてみますので、もし今後 chromatic / Deploy Storybook to chromatic (push) が失敗したらGitHubActionsの結果画面から再度Workflowを再実行してみて頂けますか🙇
kuniyuki-f
commented
1 year ago @keitakn
この件はちょっと調べてみますので、もし今後 chromatic / Deploy Storybook to chromatic (push) が失敗したらGitHubActionsの結果画面から再度Workflowを再実行してみて頂けますか🙇
ありがとうございます、承知しました！
一旦デプロイが失敗する問題は再実行により回避できたということで、お手隙でレビューいただけますと幸いです！ どうぞよろしくお願いいたします🙇
issueURL
57
この PR で対応する範囲 / この PR で対応しない範囲
Storybook の URL、 スクリーンショット
https://63d52217f1430a5ad69846cd-idopnirikl.chromatic.com/?path=/story/components-completetaskbutton-completetask--default
変更点概要
101 で実装した Next API Routes を経由するリクエスト処理を用いてタスク完了機能を実装しました。
なお、今回の実装で終了ボタンをタスクアイテムコンポーネントから切り出して、終了ボタンコンポーネントを実装する形にしています。
レビュアーに重点的にチェックして欲しい点
今のタスクアイテムコンポーネント（
MeasuringTaskItem/StoppedTaskItem）は現状役割を持ちすぎているように感じました👀 なので、タスクを終了する機能を実装すると同時にそのUIを別のコンポーネントに切り出しました。この方針で問題なさそうであれば、以下のコンポーネントを実装するイシューを作成し、対応していこうと思います！
また、これを機に以前言及されていた
MeasuringTaskItem/StoppedTaskItemを統合し、TaskItemにまとめてしまおうかと考えています！（これは別イシュー#81 で対応）上記について、もしご意見あればお伺いしたいです🙏 どうぞよろしくお願いしますー！
補足情報
4d9628ccc807f14b721a1a357cace2c035345887 のコミットについて、chromatic のビルドがこけているようなので原因を調査する必要があります