-
Notifications
You must be signed in to change notification settings - Fork 1
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
Modify do delete #63
base: master
Are you sure you want to change the base?
Modify do delete #63
Conversation
poemn
commented
Sep 10, 2021
- add user to doDelete
- fix profile, docs
docs/schema/request/doDelete.json
Outdated
@@ -8,9 +8,13 @@ | |||
"properties": { | |||
"id": { | |||
"type": "string" | |||
}, | |||
"user": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
そうか表現使い回すならここは userId
じゃなくてオブジェクトにする必要があるのか。
goMyAlps
とかは id だけ渡す形式になってるので、どっちにするか統一した方が良さそうですね。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"$ref": "../def/user.json#/properties/id"
で id だけ呼び出す方式に