Skip to content

Conversation

@fuji44
Copy link
Contributor

@fuji44 fuji44 commented Nov 16, 2022

概要

戻りのペイロードがないPOSTレスポンスでAPI側が204を返却するための対策と、開発を行いやすくするために、実行時に環境変数でリクエスト先APIを変更できるようにします。

本題とは異なりますが、READMEの記述を修正しています。

テスト

  • Get, Delete, Post, Putの各操作で想定通りの結果となること

@fuji44 fuji44 requested a review from ueno-k-work November 16, 2022 04:29
@fuji44 fuji44 self-assigned this Nov 16, 2022
Comment on lines +83 to +84
elif r.status_code == 204:
return
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

戻りのペイロードがないPOSTレスポンスでAPI側が204を返却するための対策

@fuji44 fuji44 marked this pull request as ready for review November 16, 2022 04:35
@fuji44 fuji44 changed the title Base URL can be changed Fix POST response control without payload and Base URL can be changed Nov 16, 2022
Copy link
Contributor

@ueno-k-work ueno-k-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ueno-k-work ueno-k-work merged commit f67a04b into main Nov 18, 2022
@fuji44 fuji44 deleted the feature/improved-local-dev branch November 18, 2022 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants