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

フロントエンドへの通知(再)細かい改良を加えた #54

Closed h-takeyeah closed 3 years ago

h-takeyeah commented 3 years ago
equal-l2 commented 3 years ago

@Stroheim001 マージしちゃって大丈夫でしょうか? SSEをNuxtにねじ込む実装を今やっているので、APIが変わるこのPRがdevelopに入らないと身動き取れないので……

ghost commented 3 years ago

@equal-l2 食事をとってました。すみません。

やむを得ない対応なら仕方ないと思いますが、websocketにしておけば...ってことになってなければいいですけど

APIとフロントの密度が一時的に高まるのが暫定的措置ならいいと思いますが、極力分けるつもりでいてください。

必要ならmilestoneを使って将来的に結合度を下げてテスティングしやすくする事も視野に入れてほしいです。

ghost commented 3 years ago

@h-takeyeah @equal-l2 これ、だいぶ前にも言いましたけど、とりあえず仕上がったらこれでごりおすしかないんですよね?

h-takeyeah commented 3 years ago

ゴリ押すというのは?

ghost commented 3 years ago

@h-takeyeah @equal-l2 結合が起きてしまい、API的な振る舞いをフロントエンドにも行うことを意味するんですよね?

で、それしか方法がないのなら、やむを得ないのですが、将来的にリファクタリングを予定していて欲しいです。