Skip to content
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

モデレーション画面で送れる個人向けお知らせにダイアログを指定した際、streamによる即時表示が機能しない #14258

Closed
1 task done
tai-cha opened this issue Jul 18, 2024 · 3 comments · Fixed by #14260
Labels
🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR

Comments

@tai-cha
Copy link
Contributor

tai-cha commented Jul 18, 2024

💡 Summary

タイトル通りですが、ユーザーを指定したお知らせでダイアログ形式を設定すると即時表示されません

🥰 Expected Behavior

  • WebSocketが切れていない場合は即時お知らせのダイアログが出る

🤬 Actual Behavior

  • リロードしないと出ない

📝 Steps to Reproduce

  1. 管理者またはモデレーターアカウントを用意する
  2. モデレーション画面で自身を開き、お知らせを選択
  3. ダイアログ形式でお知らせを発行
  4. リロードするまでダイアログが出ない

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey: 2024.7.0-beta.1-commit.25714+e716c201c

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@tai-cha tai-cha added the ⚠️bug? This might be a bug label Jul 18, 2024
@tai-cha
Copy link
Contributor Author

tai-cha commented Jul 18, 2024

ちなみにユーザー指定のないお知らせ(普通のお知らせ)を発行した場合には想定通り即時ダイアログが表示されます

@tai-cha
Copy link
Contributor Author

tai-cha commented Jul 18, 2024

betaが原因かと思ったけど2024.5.0でも再現した

@tai-cha tai-cha added 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR and removed ⚠️bug? This might be a bug labels Jul 18, 2024
@tai-cha
Copy link
Contributor Author

tai-cha commented Jul 18, 2024

普通のお知らせはglobalのbroadcastに流れてるけど個人向けのお知らせはpublishMainStreamで流れてるからチャンネルが違うんだ(つまり購読が未実装?)

@tai-cha tai-cha added packages/frontend Client side specific issue/PR and removed packages/frontend Client side specific issue/PR labels Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR
Projects
None yet
1 participant