Closed kuniyuki-f closed 11 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Updated (UTC) |
---|---|---|---|
timelogger-web | ✅ Ready (Inspect) | Visit Preview | Oct 8, 2023 0:45am |
今回、コードへのコメント記入はありません。 お手隙でレビューをお願いいたします!
issueURL
118
この PR で対応する範囲 / この PR で対応しない範囲
useTask
)を実装するStorybook の URL、 スクリーンショット
変更点概要
131 で実装済みのタスク計測開始ボタンも含め、カスタムフックとして
useTask
を実装しました。この
useTask
にタスクの作成、停止、再開、終了のロジックを閉じ込め、APIへのリクエスト処理やタスクUIへのレスポンスデータ反映もその中で処理しています。レビュアーに重点的にチェックして欲しい点
今回、カスタムフックとして
useTask
を実装しましたが、この実装方針についてもし気になることなどあればご指摘いただきたいと考えています。 元々 #131 を実装した時はpages/timer.tsx
の中でロジックを実装しようと考えていましたが、以下2点を考慮した結果カスタムフックの実装に転換しました。補足情報
とくになし