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

DMを送受信するときの挙動に問題がある #7655

Open
tkmrgit opened this issue Aug 18, 2021 · 10 comments
Open

DMを送受信するときの挙動に問題がある #7655

tkmrgit opened this issue Aug 18, 2021 · 10 comments
Labels
🐛Bug Unexpected behavior ✨Feature This adds/improves/enhances a feature

Comments

@tkmrgit
Copy link

tkmrgit commented Aug 18, 2021

💡 Summary

Misskey v12でv11までに利用できた宛先を指定しなくてもDMを送信する機能が使用できなくなっているようです。また、この方法で送った際の表示がおかしな状態になるようです。 参照 : https://misskey.dev/notes/8pl75fxye3

🙂 Expected Behavior

v11の時の方法で正しく送信・受信できるようにする (Mastodonもこのような形式で送ることができる)
例 : @hoge にゃーん

または、

v12 Webクライアントから送信時にDMで宛先未指定の場合にDMを送ろうとした際に警告を表示するようにする

☹️ Actual Behavior

DMが正しく送受信されていない(前述)

📝 Steps to Reproduce

  1. @hoge にゃーんのような形式でDMを送信する
  2. 受信状態を確認する

📌 Environment

Misskey v12.88.0

@tkmrgit tkmrgit added the ⚠️bug? This might be a bug label Aug 18, 2021
@tkmrgit tkmrgit changed the title DMを送信するときの挙動に問題がある DMを送受信するときの挙動に問題がある Aug 18, 2021
@tamaina tamaina added 🐛Bug Unexpected behavior and removed ⚠️bug? This might be a bug labels Aug 19, 2021
@mei23
Copy link
Contributor

mei23 commented Aug 21, 2021

公開範囲がダイレクトの場合のみ、メンションはメンションの役割をせず個別宛先指定する必要がある。
という仕様を意図してるはずで…

DBにはそれなりに入るんだけど
クライアント向けの送信は意図通り動いてなさそうで
メンション該当ユーザーに通知されないはずなのが通知されちゃったりしてるみたい。

(バグをなおすのもありだけど、わかりづらい仕様でもあるので)
普通にメンションもDMの宛先扱いでいいのではと。

@syuilo
Copy link
Member

syuilo commented Aug 21, 2021

v12 Webクライアントから送信時にDMで宛先未指定の場合にDMを送ろうとした際に警告を表示するようにする

これ実装してたはず

@syuilo
Copy link
Member

syuilo commented Aug 21, 2021

image

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2021

  1. DMの宛先とメンションを分けられる仕様なはずなのに、メンションされるべきではない人にメンションが飛んでしまう点はBug。
  2. メンションでDM出来ないのはいちおう仕様だから、これを変えたいならFeature request。

なので、どっちにしたいか決まらないと多分変わらない。

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2021

あと、DMの宛先とメンションを分けられる 仕様は連合経由でちゃんと動いてるかあやしいかも。

@mei23 mei23 added the ✨Feature This adds/improves/enhances a feature label Aug 21, 2021
@syuilo
Copy link
Member

syuilo commented Aug 21, 2021

あと、DMの宛先とメンションを分けられる 仕様は連合経由でちゃんと動いてるかあやしいかも。

他の実装が宛先とメンション分けられないようなら諦めてMisskeyもメンションも宛先として扱うようにしないとダメか~
その時は逆に「メンションしたユーザーも見れます」のような警告表示するかね

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2021

AP的にはaudienceとmentionは別物になってるから、見れるユーザーとメンションユーザーを分けて連合することは出来なくはないかも。
でも、他の実装 (Missskeyもふくめて) がちゃんと区別してるかあやしいかも。

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2021

ローカル分をDBに格納する時点でDM宛先はメンション扱いにしてたりして、なんかややこしそうだったわ。

if (data.visibility == 'specified') {
if (data.visibleUsers == null) throw new Error('invalid param');
for (const u of data.visibleUsers) {
if (!mentionedUsers.some(x => x.id === u.id)) {
mentionedUsers.push(u);
}
}
if (data.reply && !data.visibleUsers.some(x => x.id === data.reply!.userId)) {
data.visibleUsers.push(await Users.findOneOrFail(data.reply.userId));
}
}

挙動ちゃんとなおすの大変そうだから、もうメンションと宛先一緒にしちゃいたいかも。

@tkmrgit
Copy link
Author

tkmrgit commented Aug 21, 2021

v12 Webクライアントから送信時にDMで宛先未指定の場合にDMを送ろうとした際に警告を表示するようにする

これ実装してたはず

これ原因不明にゃのだが私の環境で警告出てこず送信できちゃう

@mei23
Copy link
Contributor

mei23 commented Aug 21, 2021

これ原因不明にゃのだが私の環境で警告出てこず送信できちゃう

メンションとテキスト全部入力してから公開範囲を変えた時は出てこないとかあるかも。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior ✨Feature This adds/improves/enhances a feature
Projects
None yet
Development

No branches or pull requests

4 participants