操作
バグ #992
完了
フォロー通知が大量に出る
プラグイン名:
notify
ブランチ:
クラッシュする:
説明
フォロー通知に複数のアカウントが含まれている場合、全てのアカウント名が含まれている通知がそのアカウントの数だけ複製されてポップアップしてくる、と友人が言っていました。
確かにnotify.rbを見るとそうなっています。
Osamu Koga さんがほぼ8年前に更新
- ステータス を 新規 から レビュー待ち に変更
- 担当者 を toshi_a 初音 にセット
- 進捗率 を 0 から 100 に変更
本来は1フォローごとに1つのポップアップを出すのが正しいんじゃないかと思いますが、i18nのキー名を変えないと文字列として変になってしまうのと、アンフォロー通知では全てのアカウント名を含む通知を1つだけ出すような挙動になっていたことを鑑みて、フォロー通知もこの挙動に寄せて修正しています。
toshi_a 初音 さんがほぼ8年前に更新
問題は把握しました。しかし修正方法が難しいですね。
おさけーさんが指摘しているとおり、通知をまとめるか別々にするかは意見が別れるところです。
一度ntddkがブロックされまくっているのをからかっていたら1000人くらいから一気にフォローされたことがあったのですが、そういうことがあった場合は、通知がまとまるのではなく、画面が通知で埋まるべきだと思います。とすると、まとめるのは良くなさそうです。
しかし現実的には、mikutterは今のところ、Streaming APIで受けたフォロー通知はまとめずに都度送るようになっているので、どちらの実装だろうが、通知で画面が埋まることになります。
今回はこれをmergeして、今後Streamingプラグインの実装を変更することがあれば、ここをまた考えたいと思います。
操作