Closed keitakn closed 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 19, 2023 6:41am |
@keitakn ご対応ありがとうございます! ちょうど自分も chromatic のデプロイが Error になっていたので助かりました!
issueURL
https://github.com/commew/timelogger-web/issues/83
この PR で対応する範囲 / この PR で対応しない範囲
この PR で対応する範囲
https://github.com/commew/timelogger-web/issues/83 に記載があるようにアカウント名OpenAPIの定義から削除する為の対応を実施します。
Storybook の URL、 スクリーンショット
UI変更はないのでなし
変更点概要
タイトルの通りですが、https://github.com/commew/timelogger-web/issues/83 で不要という結論が出たのでOpenAPIの定義からアカウント名を削除しました。
以下が対象のAPI仕様書のURLになります。
https://timelogger-web-git-feature-issue83-commew.vercel.app/docs/api#/operations/getAccounts
https://timelogger-web-git-feature-issue83-commew.vercel.app/docs/api#/operations/postAccounts
https://timelogger-web-git-feature-issue83-commew.vercel.app/docs/api#/schemas/Account
レビュアーに重点的にチェックして欲しい点
@c501306014 @stkzk3110
情報共有のため、レビュアーに設定させて頂いております!軽微な変更なのと、バックエンドのブロッカーになってしまっているので先行してマージさせて頂きます🙏
@HaruyaFujimoto 遅くなって申し訳ありません、対応しました!
補足情報
このPRの内容とは直接関係ないですが、最近chromatic のWorkflowのタイムアウトが多いので以下のCommitでタイムアウトを延長しました。
https://github.com/commew/timelogger-web/pull/107/commits/61740b3cb2a5cfb655f3c374ffb42ed8a5797468
ちなみにここだけでなく私の個人プロジェクトや自分が仕事で参加しているプロジェクトでも同じような事が起きているので、chromatic側の問題だと思われます。