Summary 現状、スレッドをミュートはリプライ通知とメンション通知のみを抑制しているようですが、RN通知や引用通知、リアクション

(Partially?) Duplicate of: <a class="issue-link js-issue-link" data-error-text="Failed

リプライ作成時に通知をwebhookを抑制しているっぽい? <div cla

スレッドをミュートでRNやリアクションの通知も抑制して欲しい,about misskey-dev/misskey

Sayamame-beans commented on August 15, 2024 1

これMisskeyではどちらにしましょう…

from misskey.

tamaina commented on August 15, 2024 1

upvote

RN/リアクション通知だけ消す「スレッドミュート弱」とすべての通知を発生させないとかの「スレッドミュート強」があると嬉しいかも?

from misskey.

Sayamame-beans commented on August 15, 2024

(Partially?) Duplicate of: #8102

from misskey.

Sayamame-beans commented on August 15, 2024

This is intended because thread mute is a feature that mutes conversations.
If you want to mute things like voting in polls, I think it needs to be implemented as a separate feature.

この方針は今も変わらないでしょうか…?
投稿/会話をミュートしたら、関連する引用やリアクションの通知も来ない方が自然だと思っていました
(未確認: Twitterはそうなっていた?)

from misskey.

Sayamame-beans commented on August 15, 2024

リプライ作成時に通知をwebhookを抑制しているっぽい?

misskey/packages/backend/src/core/NoteCreateService.ts

Lines 629 to 639 in efb0429

    
           if (!isThreadMuted) { 
        
           	nm.push(data.reply.userId, 'reply'); 
        
           	this.globalEventService.publishMainStream(data.reply.userId, 'reply', noteObj); 
        
           	const webhooks = (await this.webhookService.getActiveWebhooks()).filter(x => x.userId === data.reply!.userId && x.on.includes('reply')); 
        
           	for (const webhook of webhooks) { 
        
           		this.queueService.userWebhookDeliver(webhook, 'reply', { 
        
           			note: noteObj, 
        
           		}); 
        
           	} 
        
           }

それと、メンション一覧(通知ページの"あなた宛て")では表示が除外されているっぽい

misskey/packages/backend/src/server/api/endpoints/notes/mentions.ts

Line 78 in efb0429

this.queryService.generateMutedNoteThreadQuery(query, me);

おまけにinsertNoteUnread(あんまり分かっていない)

misskey/packages/backend/src/core/NoteReadService.ts

Line 58 in efb0429

if (isThreadMuted) return;

from misskey.

Sayamame-beans commented on August 15, 2024

TLに流れるのは抑制すべきか不明…

from misskey.

Sayamame-beans commented on August 15, 2024

~~新規ノート通知も抑制されていなさそう~~ 新規ノート通知はリプライを含んでいないようなので、引用リノートのみ対象となる。

misskey/packages/backend/src/core/NoteCreateService.ts

Lines 539 to 554 in 337b42b

    
           if (data.reply == null) { 
        
           	// TODO: キャッシュ 
        
           	this.followingsRepository.findBy({ 
        
           		followeeId: user.id, 
        
           		notify: 'normal', 
        
           	}).then(followings => { 
        
           		if (note.visibility !== 'specified') { 
        
           			for (const following of followings) { 
        
           				// TODO: ワードミュート考慮 
        
           				this.notificationService.createNotification(following.followerId, 'note', { 
        
           					noteId: note.id, 
        
           				}, user.id); 
        
           			} 
        
           		} 
        
           	}); 
        
           }

ただ、純粋にRN/引用をthread mute対象にするコードを組んだだけだと、返信かつ引用のケースで、その片方がthread mute対象の場合のケースを厳密に処理出来ない(ミュートされない)ので、新規ノート通知通知も抑えない方が挙動が揃うことになる。

返信かつ引用のケースで、返信元xor引用元がthread mute対象の場合に抑制しようと思うと、返信や引用を処理する前に両方あるケース用の処理を用意する必要がありそう?
新規ノート通知のことを考えるとそれより前に置きたい(新規ノート通知を後ろに置きたい)気もするが、安直に後ろに置くと例外throwを踏んだ場合にこれまでギリギリ処理されていた部分が処理されなくなってしまいそうで悩む

from misskey.

Sayamame-beans commented on August 15, 2024

途中でthrowされるケースはあまり想定しなくても良いかも?
(表に見える範囲でthrowされるのはspecifiedでvisibleUsersがnullのケースで、新規ノート通知は他人に対して有効なものかつspecifiedは除外されてるっぽいので、被らない。元々そこまで想定されてなさそうというのもある…?)

from misskey.

Sayamame-beans commented on August 15, 2024

TODO: Twitterの「この会話をミュート」の挙動を確認する

from misskey.

Sayamame-beans commented on August 15, 2024

Twitterでは

他の関係のないツイートにおいて、当人への返信の中でミュート対象のツイートを引用すると、そのツイートは通知されない
ミュート対象のツイートへのリプライにおいて、他の関係ない当人のツイートを引用すると、その引用は通知されない

このような、返信/メンションによる通知と引用による通知が同時に発生条件を満たした場合に、どちらかがミュート条件を満たしているとき通知を抑制すべきか否かは若干諸説ある

from misskey.

Sayamame-beans commented on August 15, 2024

他の関係のないツイートにおいて、当人への返信の中でミュート対象のツイートを引用すると、そのツイートは通知されない

ミュート対象のツイートへのリプライにおいて、他の関係ない当人のツイートを引用すると、その引用は通知されない

この2つのケースでは、「関係ない投稿に関する通知でもあるので見えて欲しい」という考え方と、「そのようにすれば迂回出来てしまうからやはり通知されるべきではない」という2つの考え方が発生します。

どちらか片方に寄せるのが難しければ、アカウント単位の設定で挙動を切り替えられるようにするとかになりますが…そちらの方が良いでしょうか? (それはそれで設定項目の文言が難しいですが)

追記: 設定で挙動を切り替えられるようにする予定

from misskey.

Sayamame-beans commented on August 15, 2024

それと、メンション一覧(通知ページの"あなた宛て")では表示が除外されているっぽい

misskey/packages/backend/src/server/api/endpoints/notes/mentions.ts

Line 78 in efb0429

this.queryService.generateMutedNoteThreadQuery(query, me);

後からスレッドミュートを行っても一般の通知欄では既にある通知は消えないが、メンション一覧では後からでも消えるという挙動の差はそのままで良いのか不明

from misskey.

Sayamame-beans commented on August 15, 2024

RN/リアクション通知だけ消す「スレッドミュート弱」

なるほど…
とすると、引用は弱では阻止せず、強では軒並み(私が挙げたケースでも)弾くという2択ですかね?

from misskey.

tamaina commented on August 15, 2024

RN/リアクション通知だけ消す「スレッドミュート弱」

スレッドミュートとは関係なく「RN/リアクション通知だけ消す」機能をつければいいのでは

(今のスレッドミュートの効能を勘違いしてた(嫌な話題を見ないようにするものだったわね

from misskey.

Sayamame-beans commented on August 15, 2024

ふむ…?

from misskey.

tamaina commented on August 15, 2024

今のスレッドミュートの効能

自分の投稿の通知がうるさいから設定するとかではなくて、第三者どうしが話してる不快な会話を消すためのものだったような気がする

from misskey.

Sayamame-beans commented on August 15, 2024

This is intended because thread mute is a feature that mutes conversations.
If you want to mute things like voting in polls, I think it needs to be implemented as a separate feature.

この方針は今も変わらないでしょうか…? 投稿/会話をミュートしたら、関連する引用やリアクションの通知も来ない方が自然だと思っていました ~~(未確認: Twitterはそうなっていた?)~~ (Twitterはそうなっている)

これメンションするの忘れてたみたいです…
@syuilo どうでしょうか。"スレッドをミュート"とは別の機能として分離した方が良い感じでしょうか?

from misskey.

スレッドをミュートでRNやリアクションの通知も抑制して欲しい about misskey HOT 17 OPEN

Comments (17)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	if (!isThreadMuted) {
	nm.push(data.reply.userId, 'reply');
	this.globalEventService.publishMainStream(data.reply.userId, 'reply', noteObj);

	const webhooks = (await this.webhookService.getActiveWebhooks()).filter(x => x.userId === data.reply!.userId && x.on.includes('reply'));
	for (const webhook of webhooks) {
	this.queueService.userWebhookDeliver(webhook, 'reply', {
	note: noteObj,
	});
	}
	}

	if (data.reply == null) {
	// TODO: キャッシュ
	this.followingsRepository.findBy({
	followeeId: user.id,
	notify: 'normal',
	}).then(followings => {
	if (note.visibility !== 'specified') {
	for (const following of followings) {
	// TODO: ワードミュート考慮
	this.notificationService.createNotification(following.followerId, 'note', {
	noteId: note.id,
	}, user.id);
	}
	}
	});
	}