Closed h-takeyeah closed 3 years ago
PR名を、PRの内容がわかるように変更してください。
@h-takeyeah これは青シャツさんの言う通りで、これ何?ってなりました。最低限もしissueと紐づいてるなら fix/17
にするとかが良いかと思います。PR名も命名規則を付けたほうが良いかもしれませんね...
逆に、descriptionのやる気と真反対なのはすげえなぁって思ってしまいましたが...
@equal-l2 @Stroheim001 言い忘れてましたがバックエンドから投げられてくるのはJSON.stringify()した文字列です.
了解です
@equal-l2 @Stroheim001 SSEのエンドポイントは/v1/users_updated_event
になります. この会話の最初の方に出した実装例から名前が変更になっています.
複数形,過去形で分かりにくいと思うのでメモしておきます.
Server-Sent eventsで入室および退室イベントが発生した時にフロントエンドに在室者の配列を投げます。
クライアントサイドではこんな感じでデータが取得できます.JavaScriptの例です.
まだ実現できていないのは
user_name
プロパティ(メンバーを登録する・読み出すAPIが生えてないのでまだ無理.作るまで全員りんごアイドルになってもらう)在室人数を示すプロパティが無い(付けろと言われればすぐ付けられる) です.他にも思い出したら付け足します.レビューお願いします.
close #17