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

ダイレクトでメンションでもユーザーを指定できるように #3722

Merged
merged 1 commit into from
Dec 22, 2018

Conversation

mei23
Copy link
Contributor

@mei23 mei23 commented Dec 22, 2018

Summary

Resolve #3718

この仕様にすると #3719 #3716 がいらなくなる気がする。

@tamaina
Copy link
Member

tamaina commented Dec 22, 2018

この仕様にすると #3719 #3716 がいらなくなる気がする。

つまり[+ユーザーを追加]自体要らないということ…?

@syuilo
Copy link
Member

syuilo commented Dec 22, 2018

この仕様だとメンションが意図せずユーザー指定になっちゃうことがありそう(そういうケースは実際には少ないでしょうが)

@mei23
Copy link
Contributor Author

mei23 commented Dec 22, 2018

この仕様だとメンションが意図せずユーザー指定になっちゃうことがありそう

Mastodonとかでもある誤爆なのでまあいいかなレベルに思ってるわ
どうしても困るなら #3721 と併用できるようにするとか

@syuilo syuilo merged commit c9c2853 into misskey-dev:develop Dec 22, 2018
@syuilo
Copy link
Member

syuilo commented Dec 22, 2018

ありがとうございます 🙏

@mei23
Copy link
Contributor Author

mei23 commented Dec 22, 2018

@tamaina
このプルリクの仕様だと「ユーザーを追加」が0人でも投稿を許可することになるので
#3719 と相反する感じ

メンションでも出来るようになれば「ユーザーを追加に気づきませんでした」と言われることがなくなるのでもう #3716 はさほど重要じゃないのかなと

@tamaina
Copy link
Member

tamaina commented Dec 22, 2018

@mei23

このプルリクの仕様だと「ユーザーを追加」が0人でも投稿を許可することになるので
#3719 と相反する

リプライもメンションもユーザーを追加もせずダイレクト投稿した場合、このPR以後はサーバー側で出ていたエラーも出なくなるので、宛先が虚空になる事故が発生します。なのでむしろ #3719 のようにクライアント側で補う必要があると思うのですがどうでしょう?

#3716 はUI的なもので私もさほど重要じゃないとは思っています

@mei23
Copy link
Contributor Author

mei23 commented Dec 22, 2018

@tamaina
今までと同じように投稿に失敗するようにしとけばいいかなと #3724

@tamaina
Copy link
Member

tamaina commented Dec 22, 2018

@mei23
👍

@mei23 mei23 deleted the mei-1223-direct-mention branch December 30, 2018 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants