-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
|
…j-marowd into refactor_backend_fix
swaggerを元に戻した。
コメント全て読みました。ファイルの切り分けで何やってるかがスッキリしました |
|
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.
LGTM
@@ -32,7 +32,7 @@ export const GetClassDataHandler = async ( | |||
} catch (exception) { | |||
console.log("class not found probably wrong or empty query"); | |||
console.log(exception); | |||
resp.sendStatus(404).send("Not Found"); | |||
resp.sendStatus(404); |
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.
resp.sendStatus(404)で主にレスポンス送ってるのにさらに.send("Not Found");って書いてるのがダメだったぽいね!
一括で書き換えしてたので修正漏れていました。ありがとうございます!
return; | ||
} | ||
resp.sendStatus(200).send(JSON.stringify(qss.data())); | ||
resp.send(JSON.stringify(qss.data())); |
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.
@kanade9
この場合、ステータスコードって200になってる?
なるならそのままでおkですが、ならないなら
resp.status(200).send(JSON.stringify(qss.data()));
にする必要がありそうです。確認お願いします!
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.
修正しました!
修正したのでマージします!! |
TODO