misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
10.12k stars 1.38k forks source link

新規投稿通知がリノートミュートを考慮しない #14925

Closed kakkokari-gtyih closed 1 week ago

kakkokari-gtyih commented 3 weeks ago

💡 Summary

新規投稿を通知する機能があるが、リノートが(リノートミュートをしていても)通知されてしまう

あたりを考える必要があるかもしれない

Original issue: https://support.misskey.io/hc/ja/community/posts/11020080768527-%E9%80%9A%E7%9F%A5%E6%AC%84%E3%81%AB%E3%83%95%E3%82%A9%E3%83%AD%E3%82%A4%E3%83%BC%E3%81%8C%E3%83%AA%E3%83%8E%E3%83%BC%E3%83%88%E3%81%97%E3%81%9F%E9%80%9A%E7%9F%A5%E3%82%92%E9%9D%9E%E8%A1%A8%E7%A4%BA%E3%81%AB%E3%81%97%E3%81%9F%E3%81%84

🥰 Expected Behavior

以下のどれか

🤬 Actual Behavior

リノートが(リノートミュートをしていても)通知されてしまう

📝 Steps to Reproduce

  1. AがBをフォローして投稿通知を有効にする
  2. AがBをリノートミュートする
  3. Bが誰かのノートをリノートする
  4. 通知される

💻 Frontend Environment

* Misskey: (vanila develop)

🛰 Backend Environment (for server admin)

* Misskey: (vanila develop)

Do you want to address this bug yourself?

penginn-net commented 2 weeks ago

個人的には通知にリノートを含むかを変えられたらいいと思います

Sayamame-beans commented 2 weeks ago

リノートミュートは見るの忘れてたな… 含むか分けられるとなお良さそうなのはそれとして(別issue開きます?)、通知しないようにするべきですね (off-topic: 近い話だと、スレッドミュートで新規投稿通知対応しないといけないことに気付いたのがあったのですが、実装が途中から進められていません…)