traP-jp / traO-Judge-backend

traO Judge バックエンド
MIT License
6 stars 0 forks source link

Feat/#9 get users userId #34

Closed shobonvip closed 1 month ago

shobonvip commented 1 month ago

関連Issue

概要

get users/{userId} の実装

変更内容

補足

shobonvip commented 1 month ago

formatチェック機能とかあるんだ

shobonvip commented 1 month ago

try_get_user_by_id を get_user_by_id に統合します このとき get_user_by_id 関数を使っている get_me と put_me において user が見つからなかったときの処理が必要になるので、それぞれ 1 行追加します(ほとんどありえない場合分けだが、 INTERNAL_SERVER_ERROR とした)

kenken714 commented 1 month ago

(これはこのPRより前から残っていた問題なのですが) user の icon_url がDBスキームでは nullable であるのに対し、Userの構造体では許容されていないため、get_user_by_idでエラーが出てしまいました src/repository/users.rs の 22行目 pub icon_url に Option を付けてほしいです

kenken714 commented 1 month ago

(マージしていただいて大丈夫です!)