sakaba-project / sakaba

0 stars 0 forks source link

Add db connection #40

Closed unvavo closed 4 years ago

unvavo commented 4 years ago

https://github.com/sakaba-project/sakaba/issues/27 のやつ

https://github.com/sakaba-project/sakaba/pull/25 取り込んでます

やったこと

レビュー観点

動作確認

コンテナ起動して下記実施してみる

GET http://127.0.0.1:8080/users (一覧取得)
POST http://127.0.0.1:8080/users (追加)
GET http://127.0.0.1:8080/users/:id (個別取得)
DELETE http://127.0.0.1:8080/users/:id (削除取得)
ghost commented 4 years ago

dbinit 周りだいぶ綺麗になってた!ありがとう! いくつか質問書いた

ghost commented 4 years ago

ノブ対応ありがとう!

個人の好みレベルの指摘を入れたので なんか良さそうと思えば変更してくれればって感じです。

基本的にコード全体わかりやすかったです!LGTM

ghost commented 4 years ago

LGTM マージしました