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