Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

openapi.yamlへのrequiredの追加 #2

Merged
merged 5 commits into from
Feb 1, 2024
Merged

Conversation

eyemono-moe
Copy link
Collaborator

型情報の修正のためrequiredを一部追加しました
PatchGameRequestについてはそもそもidがプロパティに含まれていなかったので追加しました
不都合等あったら教えてください

(ダブルクォートとシングルクォートが混在していたのでシングルクォートに統一しちゃいました)

Copy link
Collaborator

@pikachu0310 pikachu0310 left a comment

Choose a reason for hiding this comment

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

大体大丈夫そうです、ありがとうございます。
1件確認なのですが、Patchにidを含めるのってあんまり一般的ではないような気がしています。(IDは通常不変で、リソースを特定するためにURLで既に使用されている)
idを変更したいという需要があれば追加しますが、無ければ無くても良いと思います。どうでしょうか?

@eyemono-moe
Copy link
Collaborator Author

pathにID入ってるの完全に失念してました、そこだけ削除しました

@eyemono-moe eyemono-moe merged commit b0f4763 into main Feb 1, 2024
2 of 4 checks passed
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.

2 participants