Closed stkzk3110 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 | Apr 17, 2023 11:33pm |
keitakn
commented
1 year ago @stkzk3110
PR内の各箇所にコメント残してますのでそこら辺を重点的に見てもらえればと思います。
この件ですがインラインコメントではなくソースコード内のコメントというお話であれば確認済です:+1:
stkzk3110
commented
1 year ago @keitakn ありがとうございます！ そうですね、まだ最終調整があったので、submit前でした。。。
今回もらった指摘も含めて調整終わったら再度レビュー依頼投げさせてもらいますね🙏
issueURL
https://github.com/commew/timelogger-web/issues/42
この PR で対応する範囲 / この PR で対応しない範囲
https://github.com/commew/timelogger-web/issues/42 の完了の定義を満たしていること。
APIのレスポンス結果を渡して表示するところは一旦、mockのdataを用意して固定値で表示するようにしてます。APIのレスポンス結果に変える対応は別PRで対応します。 テストコードも一旦、mockを使用しているため、観点だけ記載してます。APIの結果反映時に別PRで対応します。
Storybook の URL、 スクリーンショット
https://user-images.githubusercontent.com/36589062/230774109-1614a93a-cfa2-4751-b9d3-2b8dc04e86f7.mov
変更点概要
サイドバーコンポーネントを作成しました。
レビュアーに重点的にチェックして欲しい点
PR内の各箇所にコメント残してますのでそこら辺を重点的に見てもらえればと思います。
補足情報
特になし。