プロジェクト

全般

プロフィール

バグ #992

フォロー通知が大量に出る

Osamu Koga10ヶ月前に追加. 10ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
notify

説明

フォロー通知に複数のアカウントが含まれている場合、全てのアカウント名が含まれている通知がそのアカウントの数だけ複製されてポップアップしてくる、と友人が言っていました。
確かにnotify.rbを見るとそうなっています。

関係しているリビジョン

リビジョン f7ef4479 (差分)
Osamu Koga10ヶ月前に追加

フォロー通知は一度に1つしか出ないようにした refs #992

履歴

#1 Osamu Koga10ヶ月前に更新

  • ステータス新規 から レビュー待ち に変更
  • 担当者toshi_a 初音 にセット
  • 進捗率0 から 100 に変更

本来は1フォローごとに1つのポップアップを出すのが正しいんじゃないかと思いますが、i18nのキー名を変えないと文字列として変になってしまうのと、アンフォロー通知では全てのアカウント名を含む通知を1つだけ出すような挙動になっていたことを鑑みて、フォロー通知もこの挙動に寄せて修正しています。

#2 toshi_a 初音10ヶ月前に更新

問題は把握しました。しかし修正方法が難しいですね。
おさけーさんが指摘しているとおり、通知をまとめるか別々にするかは意見が別れるところです。
一度ntddkがブロックされまくっているのをからかっていたら1000人くらいから一気にフォローされたことがあったのですが、そういうことがあった場合は、通知がまとまるのではなく、画面が通知で埋まるべきだと思います。とすると、まとめるのは良くなさそうです。
しかし現実的には、mikutterは今のところ、Streaming APIで受けたフォロー通知はまとめずに都度送るようになっているので、どちらの実装だろうが、通知で画面が埋まることになります。

今回はこれをmergeして、今後Streamingプラグインの実装を変更することがあれば、ここをまた考えたいと思います。

#3 toshi_a 初音10ヶ月前に更新

  • ステータスレビュー待ち から 終了 に変更

hotfix/3.5にmergeしました。

他の形式にエクスポート: Atom PDF