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

投稿ごとのミュート #2227

Open
syuilo opened this issue Aug 15, 2018 · 14 comments
Open

投稿ごとのミュート #2227

syuilo opened this issue Aug 15, 2018 · 14 comments
Assignees
Labels
[Feat] Mute/Block Mute or Block related issue. It does not cover server blocking or server muting. ✨Feature This adds/improves/enhances a feature

Comments

@syuilo
Copy link
Member

syuilo commented Aug 15, 2018

Summary

Environment


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@syuilo syuilo added the ✨Feature This adds/improves/enhances a feature label Aug 15, 2018
@tamaina
Copy link
Contributor

tamaina commented Aug 15, 2018

#2144 に似ている気がしなくもない

@tamaina
Copy link
Contributor

tamaina commented Dec 19, 2022

(クライアント側が検出して)一定のRenote数があったら一定期間(引用を除いて)Renoteをミュートするみたいな機能欲しいけど、引用を含むか含まないかのフラグが必要か

@samunohito
Copy link
Member

@syuilo
要件的にどちらでしょう…?

  • ノートを非表示にしたい(タイムライン取得の各種APIに該当のノートを含めないようにしたい)
  • ノートの通知を切りたい(該当ノートに対する通知を全部オフにしたい)

@syuilo
Copy link
Member Author

syuilo commented Feb 12, 2025

前者だわね

@syuilo
Copy link
Member Author

syuilo commented Feb 12, 2025

パフォーマンス的に難しそう

@samunohito
Copy link
Member

develop...samunohito:feat/note-mutings#diff-0ac446f371898719494ec2f72e1fe3958790bc82a9b4bd14a06b937bcc80f061

既存のユーザミュートの仕組みと似たような感じでいけませんかね

@samunohito
Copy link
Member

(厳しいなら閉じてしまっても良いかも)

@syuilo
Copy link
Member Author

syuilo commented Feb 12, 2025

いけそう
けどIssue作成が7年前なので、今の需要がどれくらいあるかは改めて調査したい

@samunohito
Copy link
Member

https://mi.osamu-storage.info/notes/a45dsv945qu700kv
(アンケ中。まだ始めて間もないけどそこそこ需要がある様子)

@samunohito
Copy link
Member

結果発表

Image

@syuilo
Copy link
Member Author

syuilo commented Feb 13, 2025

ほむん
Misskeyにとってのユーザー(=サーバー管理者)に「パフォーマンスが低下したとしてもそのような機能をユーザーに提供したいか否か」を聞きたい感あるわね

@samunohito
Copy link
Member

パフォーマンス

具体的にどのあたりを気にされていますか…?
非常に大きいクエリを追加するわけでもなく、追加された部分は大体Redisやメモリキャッシュを活かしているのでそんなに影響は無いと考えてるのですが…

@syuilo
Copy link
Member Author

syuilo commented Feb 13, 2025

例えばストリーミングを考えると、ある程度キャッシュできるとはいえ投稿が来るたびにミュートしてないかどうかのチェックが必要になりそう

@samunohito
Copy link
Member

(既にやってるような…)

// 流れてきたNoteがインスタンスミュートしたインスタンスが関わる
if (isInstanceMuted(note, new Set<string>(this.userProfile?.mutedInstances ?? []))) return true;
// 流れてきたNoteがミュートしているユーザーが関わる
if (isUserRelated(note, this.userIdsWhoMeMuting)) return true;
// 流れてきたNoteがブロックされているユーザーが関わる
if (isUserRelated(note, this.userIdsWhoBlockingMe)) return true;
// 流れてきたNoteがリノートをミュートしてるユーザが行ったもの
if (isRenotePacked(note) && !isQuotePacked(note) && this.userIdsWhoMeMutingRenotes.has(note.user.id)) return true;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feat] Mute/Block Mute or Block related issue. It does not cover server blocking or server muting. ✨Feature This adds/improves/enhances a feature
Projects
Development

Successfully merging a pull request may close this issue.

3 participants