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

通知を削除する方法を提供して欲しい #13317

Closed
1 task
Sayamame-beans opened this issue Feb 15, 2024 · 44 comments · Fixed by #13335
Closed
1 task

通知を削除する方法を提供して欲しい #13317

Sayamame-beans opened this issue Feb 15, 2024 · 44 comments · Fixed by #13335
Assignees
Labels
✨Feature This adds/improves/enhances a feature 🔥high priority

Comments

@Sayamame-beans
Copy link
Member

Summary

タイトル通りです。

Purpose

何らかの理由( #13316 関連など?)で通知を消したい場合があります。
現状、発生した通知を消す方法は存在しないため、消せるようになると良いと思います。

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@Sayamame-beans Sayamame-beans added the ✨Feature This adds/improves/enhances a feature label Feb 15, 2024
@tamaina
Copy link
Contributor

tamaina commented Feb 15, 2024

🔥

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Feb 16, 2024

こんなのでええんか(ユーザー多いと時間かかりそう)

image

@kakkokari-gtyih
Copy link
Contributor

とりあえずユーザーサイドで適宜flushできるようにしとくか

@Sayamame-beans
Copy link
Member Author

全削除ではなく個別削除が理想なので、最終的には個別削除が実装されることを望みます

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

通知を個別削除する場合、グループ化されたリアクションの削除とかどうするか悩ましいな

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

個別削除要るかしら

@samunohito
Copy link
Member

個人的にですが、むしろ全削除より個別の方が欲しいです

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

個別削除はできると嬉しい

  • 特に反応が少ないアカウントだと嫌な通知が残り続ける場合がある
  • 特定条件で削除するなどのスクリプトを組めるようになる
  • 実績や嬉しいリアクションやメンションなどの通知が消えるのは悲しい

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

そもそも削除自体要らない気がするわね

特に反応が少ないアカウントだと嫌な通知が残り続ける場合がある

嫌な通知というのがあまり想像できないけど、我慢してもらう

特定条件で削除するなどのスクリプトを組めるようになる

それやるならそもそも通知を作成する前の段階でやれるようにするべき

実績や嬉しいリアクションやメンションなどの通知が消えるのは悲しい

通知は揮発的だからそういう通知が残ることは期待してはならない

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

嫌な通知というのがあまり想像できないけど

  • ネガティブや弯曲的に嫌がらせとなりうるリアクション
  • 関係を切るほどの人間じゃない人間から飛んできたメッセージ(これも嫌な通知には個人差がある)
  • 管理者の対応が間に合わない場合のスパム
  • etc...

通知を作成する前の段階でやれるべき

通知は揮発的だから

DBに保存されていないという意味ではそうだけどそれはあまりユーザーに伝わっている気がしないしその感覚はない
Redisもバックアップとっているところがそこそこある気がする[要出典]

そのほか

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

DBに保存されていない

RedisというDBには保存されてる
ただ保存される数に制限を設けている

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

やるとしたら全削除

@Sayamame-beans
Copy link
Member Author

ただ保存される数に制限を設けている

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

もちろんユーザーからしたらずっと保存されていた方が嬉しいんだけど、管理者からするとそうではない場合がある

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

まあ全削除ならキーを削除するだけなのでパフォーマンス面の懸念は落ちる(保存されているものを読みにいく必要がない)
ただそれを踏まえても需要があるとは思っているし通知が個別削除できた方が利便性は高いと思う、天秤にかけてどちらが上になるかは完全に言い切れないけれど

@Sayamame-beans
Copy link
Member Author

管理者からするとそうではない場合がある

私個人としては、それを各サーバーの管理者が選べるように(optionalに)するために設定項目や機能が存在すると思っています…

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

個別削除できれば便利だし需要はあるけど、それをやると機能の趣旨が意図しない方に寄ってしまう

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

管理者からするとそうではない場合がある

私個人としては、それを各サーバーの管理者が選べるように(optionalに)するために設定項目や機能が存在すると思っています…

それにもコストが発生するし、機能の設計思想がブレることになる

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

機能の趣旨をどう捉えているかに齟齬がありそう

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

というよりはMisskey内での通知とスマートフォンやGitHubなどの通知の概念が少し違うから理解しずらい?(他での通知があまり揮発的なイメージがない)

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

スマホとかの通知は勝手にどんどん消えるから揮発性のイメージ

@tai-cha

This comment was marked as off-topic.

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

設計思想というより利用者のメンタルモデルと言った方が近いかも
個別に削除することができると、逆に考えると「削除しなければ残り続ける」みたいな印象を与える可能性が高まる

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

通知は消えることがある認識はもちろんあるけど逆に一定期間は保持される認識もあるのでそれを全削除か全部残すかみたいなAll or Nothingな選択肢になるのは少し違和感はある

割とお知らせでアップデート時に通知が消えることがありますの表示が書かれるのもその認識が多いからだと思っている

@anatawa12
Copy link
Member

ユーザ視点としてはスパムが目障りだから消したいに対する答えとして全部消えるは強い違和感があります

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

スパム関連であればそのユーザーをミュートなりする方が良さそう

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

ミュートしても通知消えないのでそれはそれでやった方がいい

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

ミュートしたら通知消えるようにすれば良さそう

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

ただスパム対策だけのIssueではないと思っているのでスパム対策に限った話で終わらせたくないかも

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

スパム以外では不要と思ってる

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

必要か不要かの考えは通知が流れていってあまり留まらない場合と結構通知がきて流れていく場合でも変わってきそうだと思っている(有名人に個別削除機能はあまり必要ないと思う)

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

ここまでで全く考えが変わらないのであればMisskeyのオーナーはしゅいろさんなのでMisskey自体には入れないのでもいいんじゃないという考えはある(けど議論し尽くせてはいないと思う)

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

スパム以外では不要と思ってる

個人的に必要か不要かということではなくて、現時点でソフトウェアとして当該機能は不要という判断

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

ユーザーが望む機能を、無秩序に追加していくことはできない

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

あと通知はデフォルトでグループ化されるから、「個々の通知」みたいなメンタルモデルにはなってないからUIが悩ましいことになる

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

ユーザーが望む機能を、無秩序に追加していくことはできない

これには同意なので基本的にプラグインで解決できる方法を推してはいる
その上で私はこれに関してはMisskeyに入れるメリットがあると思っていたので全くないという判断なのであればプラグインのフックを増やしたりフォークを使えばいい話になってくるかなとは思います

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

あと通知はデフォルトでグループ化されるから、「個々の通知」みたいなメンタルモデルにはなってないからUIが悩ましいことになる

これスマホの通知みたいにグループを展開できればいいのでは?

@tai-cha
Copy link
Contributor

tai-cha commented Feb 19, 2024

ミュートしたら通知消えるようにすれば良さそう

探したけど意外とIssueない?探し方が悪いかしら

@samunohito
Copy link
Member

似たようなものはありました。
#12751

…が、ユーザのミュートに追従してという切り口ではなさそうです

@samunohito
Copy link
Member

結論としては…

  • 通知を削除する機能は「設けない」
  • 別issueで考える
    • ユーザミュートを通知に反映させる
    • 通知にプラグインのhookを生やす

でOKでしょうか

@samunohito
Copy link
Member

(やらないならやらないで、closeしたい気持ちが強い)

@syuilo
Copy link
Member

syuilo commented Feb 19, 2024

全削除は実装しても良いと思う

@tamaina
Copy link
Contributor

tamaina commented Feb 19, 2024

個人的には、ミュートや凍結がしっかり反映されていれば通知を消す機会はないかなと

@tamaina
Copy link
Contributor

tamaina commented Feb 19, 2024

(FTTも全削除機能欲しい)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature 🔥high priority
Projects
None yet
7 participants