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

自分がブロックした人のリアクションをレスポンスから除外して欲しい #13456

Open
1 task
KisaragiEffective opened this issue Feb 25, 2024 · 11 comments
Labels
✨Feature This adds/improves/enhances a feature

Comments

@KisaragiEffective
Copy link
Collaborator

Summary

自分がブロックした人のリアクションをバックエンドのレスポンスから除外して欲しい

Purpose

精神衛生に悪いため

Do you want to implement this feature yourself?

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

tai-cha commented Feb 25, 2024

related: #13411

@tai-cha

This comment was marked as outdated.

@KisaragiEffective
Copy link
Collaborator Author

KisaragiEffective commented Feb 26, 2024

それでは解決しません。
私が意図していたのはノートのリアクション表示(図1)において、リアクションにブロックしているユーザーが含まれているのであればその数を表示から減じて欲しいという意味です。
例えば図1の枠線で囲ったリアクションにブロックしているユーザーが1人ずつ含まれている場合、図2のようにしてほしいです。

図1
現行のノートのリアクション表示
図2
望ましいと思うノートのリアクション表示

@syuilo
Copy link
Member

syuilo commented Feb 26, 2024

パフォーマンス的に実現不可能かも

@KisaragiEffective
Copy link
Collaborator Author

パフォーマンス的に実現不可能かも

そこなんですよね (実現しようとするといちいちブロック関係を調べる必要がある)。
代替案としてはフロントエンドでいい感じに結果を操作することも考えましたが、次の2点が懸念です。

  1. 自分がブロックしたアカウントをいちいち問い合わせる/ストリーミングしてもらうことになる。これは多分望ましくない
  2. サードパーティークライアントとの相性がよくない (多分どうにかなる)

@tai-cha
Copy link
Contributor

tai-cha commented Feb 29, 2024

ブロック済みアカウントidをstoreにキャッシュすることで一応パフォーマンス面自体の問題は大きくなくなるかも

@tai-cha
Copy link
Contributor

tai-cha commented Feb 29, 2024

ただ自分に対するリアクションじゃないならブロックよりミュートの役割な気がします

@syuilo
Copy link
Member

syuilo commented Feb 29, 2024

今はどのリアクションがいくつ付いてるかという情報しか取ってきてないから自分がブロックしているアカウントをノーコストで取れたとしても意味がないわね

@tai-cha
Copy link
Contributor

tai-cha commented Feb 29, 2024

あーリアクション(した人一覧)を押したときに出る人間を消してほしいって要望じゃなくて数を減らしてほしいだと難しいかもしれない

@Sayamame-beans
Copy link
Member

Sayamame-beans commented Feb 29, 2024

個人的には、リアクション一覧に、ユーザーミュート(ブロックではなく)した相手が載らないぐらいが丁度良い塩梅なのかなと思ったり…? (もうなっている?)

@akanevrc
Copy link
Contributor

akanevrc commented May 17, 2024

note.reactionAndUserPairCacheを利用するのはどうでしょうか?
少なくともそれぞれのリアクションのユーザーを全て検索する必要がなくなります。
実験的に実装してみましたが、最初の16人についてミュートを適用するだけでも精神衛生という観点では非常に良い結果が生まれそうです。
(リアクションが悪目立ちするのはリアクション数が少ないときである気がする(???))
これに加え、リアクション一覧からmutee userを消去すると良いと思います。

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
Projects
Status: Triage
Development

No branches or pull requests

5 participants