search

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

Iss33 readerInputControllerのエラー時にレスポンスが戻ってこないことがあるバグの修正 #39

Closed h-takeyeah closed 3 years ago

h-takeyeah commented 3 years ago

@equal-l2 @Stroheim001 #33 の修正です．今までハンドリングできていなかった以下のエラーもハンドルするようにしました．

h-takeyeah commented 3 years ago

ページネーションの実装に伴うaccessLogsController.tsの変更を取り込みました

equal-l2 commented 3 years ago

@Stroheim001

ghost commented 3 years ago

@h-takeyeah え～と、変更を取り込んだ、ということはたぶん私が指摘をして、それを取り込んだというのを

仰りたいと思うのですが、出来れば、見てほしいものがある場合、4W1Hをつけておくと非常にレビュワーとしては助かります。

以後、取り組んでもらえるとありがたいです。 覚えてない私も悪いのですが...orz

h-takeyeah commented 3 years ago

@Stroheim001 さんの指摘を取り込んだわけではなく，単にこのPRを始めた後にdevelopにpushされたコミットとこのブランチのコミットがコンフリクトして原因がaccessLogsController.tsだったのでマージしただけです．分かりにくくてすみません．以後気を付けます．

ghost commented 3 years ago

@h-takeyeah あ～、ごめんなさい。 出来れば技術系の話は時系列が分かるように話さないとアンジャッシュ状態になりかねないので、気を付けたいですね...

了解しました。