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 7, 2023 3:36pm |
keitakn
commented
1 year ago @stkzk3110
一旦、issueの通り文言などは固定値で入れてますのでログイン状態で切り替える実装は別PRで実装します。
一応補足でこのHeaderが利用されるページは完全にログインしないと閲覧出来ない仕様のようなのでここは実装不要かもですね:octocat:
kuniyuki-f
commented
1 year ago @stkzk3110 お疲れ様です！確認遅くなりすいません。
一通りPRコメント拝読し、コード・画面の表示を確認しました。 PRコメントの書き方勉強になります🙇♂️
コード・画面の表示もLGTMでした。 ご対応ありがとうございます！
issueURL
https://github.com/commew/timelogger-web/issues/41
この PR で対応する範囲 / この PR で対応しない範囲
https://github.com/commew/timelogger-web/issues/41 の完了の定義を満たしていること。
Storybook の URL、 スクリーンショット
変更点概要
HeaderNavigationコンポーネントを追加しました。 共通で参照されるところから呼び出せるように設置してます。 storybookでカスタムテーマを呼び出せるようにstorybook.preview.jsにも対応入れてます。
レビュアーに重点的にチェックして欲しい点
PR内の各箇所にコメント残してますのでそこら辺を重点的に見てもらえればと思います。
補足情報
特任あし。