-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No WebSocketモード #12227
Comments
Twitter(X)みたいに「n件の新しいツイート」の件数だけを返すのと、新しい通知があればtrue返すみたいなapiがあれば出来そうだと思った(現状がここらへんをwebsocketを使用してるため) |
@syuilo
☆1…ひとまず有無のみで考えていますが、件数のほうが良さそうでしょうか。 |
そんな感じになりそうですが、リアクションの更新をどうするかなど他にも考えなければならないことが膨大にありそうですね |
※メモ
これに、RedisのlatestReadNotificationを見る機能を増やす |
|
最新ノート取得時に、新規で取得するノートの規定数にプラスして、既に表示されているノートの最新数件(MkPaginationが持ってるdisplayLimitあたりが妥当と考えます)を取るのが良いかと思いました。新しいノートを取得しつつ、リアクションの状況、リノートやリプライなどの状態も一緒に更新できるかとおもいます。 以下のようなイメージです。
|
リアクションを考えると、5秒に1回程度は表示を更新したいけど、5秒に1回
をしてると通信量がかなり増えてしまいそうな気がした |
最新ノート取得時にまとめて更新するのではなく、リアクションの鮮度のみを重要視されるのであれば… [
{
"noteId": "9m2rp5kn45mv000r",
"reactionEmojis": {
'[email protected]' : "https://s3.arkjp.net/misskey/ceb88abf-e2dd-4e14-b226-82dc5c21b2f7.png"
},
"reactions": {
':60fpsparrot:' : 2,
'🍮' : 2,
':[email protected]:' : 334
}
},
{
"noteId": "9m30fhprh0ym0008",
"reactionEmojis": {},
"reactions": {
'🎉' : 4,
'🦄' : 1
}
}
] MkPagination内にあるノートのIDをまとめて投げる→受信結果をMkPaginationが持ってる各Noteのオブジェクトに上書き…みたいな形で更新できるかとおもいます |
良さそう |
Is there any update on this? Being able to have WebSockets be optional for timeline auto-updates, notifications, and emoji reactions would be really useful for servers behind CDNs that have no support for proxying WebSocket requests to origin... |
#13929 でWebSocketを回避する実装をやった(WebSocket未使用時の通常UIの振る舞い等は未実装) |
Summary
WebSocketを一切使わずにクライアントを利用するモード
場合によってはサーバーの負荷軽減になるかも
The text was updated successfully, but these errors were encountered: