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 26, 2023 3:11am |
keitakn
commented
1 year ago @c501306014
すいません、以下の要因から一旦レビューは待ってください🙏
こちら見落としていました、すいません💦
ちなみにこういう場合はPRをDraft状態にすれば分かりやすいかと思います👍
https://zenn.dev/inabajunmr/articles/9a683214fa32cc
（もしかして権限がないとかですかね？）
kuniyuki-f
commented
1 year ago @keitakn
レビューありがとうございます🙇
こちら見落としていました、すいません💦 ちなみにこういう場合はPRをDraft状態にすれば分かりやすいかと思います👍 https://zenn.dev/inabajunmr/articles/9a683214fa32cc
とんでもないです！ こちらこそややこしくしてしまってすいません！ またアドバイスありがとうございます！
（もしかして権限がないとかですかね？）
私が先ほど確認したところ、 既に Draft に変わってたんですが、 keita さんが変更して下さったんでしょうか？？👀
既に Draft になっているので、今の画面だと権限があるのか不明ですが、 次回から PR の状態に気を付けておくようにします！🙏 もし万一権限がなかったらむちょこさんに権限付与依頼します👍
問題が解決したらまたレビュー依頼出します。 その際はどうぞよろしくお願いします🙇
keitakn
commented
1 year ago @c501306014 ご確認ありがとうございます👌はい、先程自分の方でdraftに変えておきました！
共有してなくてごめんなさい🙏
kuniyuki-f
commented
1 year ago @keitakn
下記について無事解決しました！ ( コードの変更は 3e901cc91d54bc0e38be51c35db1adb0e3d8060a のみです！ )
Vercel のデプロイがこけている
Vercel で Redeploy したら解決しました。 ログにネットワークエラーのような記録があったので、一時的な問題だったようです。
TaskItem コンポーネントの Storybook を更新しないと時間が Nan になってしまう
3e901cc91d54bc0e38be51c35db1adb0e3d8060a で対応しました！
Draft を外しますので、お手隙で再度レビューをお願い致します！
(ちゃんと自分でもDraftに変更できそうです👀)
issueURL
81
この PR で対応する範囲 / この PR で対応しない範囲
タスクアイテムコンポーネントに時間計測ロジックを追加しました。 初回レンダリング時のステータスに応じて以下のように処理を実装しています。
今回のPRではタイマーに関する停止・終了機能は実装しません。
Storybook の URL、 スクリーンショット
変更点概要
TODO で放置していたタスクアイテムコンポーネントの時間計測ロジックを追加しました。 最初はタイマーコンポーネントみたいな形で実装しようと思っていましたが、 ロジックの要素が強かったのでカスタムフックで実装しました。
レビュアーに重点的にチェックして欲しい点
ソースコードにコメントで記載します。
補足情報
とくになし