su-its / ams-backend

:briefcase: (This repository is no longer maintained) The backend server of our Access-management-system.
MIT License
0 stars 0 forks source link

API仕様書と実際のソースコードが乖離している(表記ゆれ) #28

Closed ghost closed 3 years ago

ghost commented 3 years ago

userIdとuser_idで表記ゆれを起こしており、APIのエンドポイントから利用する側にとっては非常に不便である。

h-takeyeah commented 3 years ago

userIdの件は文法上は許容される表記ゆれでした.しかし,TSの規約的にキャメルケースで書いたとはいえ,外から入ってくるデータがスネークケースではないのは気持ち悪いのでinRoomUsersRoutes.ts内のuserIdはuser_idと書き直しました.それに伴ってinRoomUsersController.tsも修正しました.

今回のように2つのファイルを読まないと文法上許容されることが分からないのはバグのもと,いやバグだったので以後気を付けます.