commew / timelogger-web

時間記録アプリ
https://timmew.commew.net
MIT License
2 stars 0 forks source link

バックエンドのAPIが完成したので NEXT_PUBLIC_DEBUG_MOCK_API の条件分岐を削除 #142

Closed keitakn closed 11 months ago

keitakn commented 11 months ago

issueURL

https://github.com/commew/timelogger-web/issues/77

この PR で対応する範囲 / この PR で対応しない範囲

https://github.com/commew/timelogger-web/issues/77 の完了の定義を満たす NEXT_PUBLIC_DEBUG_MOCK_API に関する条件分岐を削除します。

それ以外の対応はこのPRでは実施しません。

Storybook の URL、 スクリーンショット

タスクの記録でエラーが発生する

スクリーンショット 2023-10-21 1 01 57

変更点概要

タイトルの通りです。APIの環境が完成したようなので、Vercel上の環境変数(BACKEND_API_BASE_URL)も含めて https://commew.slack.com/archives/C02JQQU4VLY/p1697705273062939 で共有された内容に変更してあります。

ローカルで確認しましたがログインは出来るが、タスクの作成で500エラーが返ってくるという状況です。

この500エラーの原因はバックエンド側の POST /tasks のステータスコードが200とAPI仕様書と異なっているためです。

https://timmew.commew.net/docs/api#/operations/postTasks

@HaruyaFujimoto @HAYASHI-Masayuki 共有しておきます。

レビュアーに重点的にチェックして欲しい点

おそらく問題ないかと思いますが、軽く確認をお願いします🙏

補足情報

特になし

vercel[bot] commented 11 months 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 Oct 20, 2023 4:07pm