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 | May 13, 2023 10:18am |
issueURL
https://github.com/commew/timelogger-web/issues/73
この PR で対応する範囲 / この PR で対応しない範囲
https://github.com/commew/timelogger-web/issues/73 のDoneの定義を満たすために必要な実装を行います。
Storybook の URL、 スクリーンショット
UI変更はないのでなし。
変更点概要
ログイン実施時にアカウント取得API、必要に応じてアカウント作成APIをコールする処理を追加しました。
ただこのままだとVercelへのデプロイ時にログインが実行出来なくなってしまうので環境変数に
NEXT_PUBLIC_DEBUG_MOCK_APIという値が1になっている時だけMockのAPIサーバーに通信するように制御しています。この処理は本番リリース時にはもちろん必要ないので https://github.com/commew/timelogger-web/issues/77 で削除する予定です。
レビュアーに重点的にチェックして欲しい点
インラインコメントに記載してある点をご確認下さい。
補足情報
ちなみにアカウント作成などに失敗した場合はログインページにリダイレクトされます。
失敗時のErrorをSentryなどに送信するようにするべきですが予算の都合上難しそうなので、この対応は一旦行わない方針で行きます。
※ 将来的に課金などが発生して商用のアプリケーションとなる場合はここのエラー通知は必ず実施する必要があると考えています。