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

ノート(リプライ(ダイレクト)・引用・メンション)を既読にしたら、通知も既読にする #7165

Closed
tamaina opened this issue Feb 9, 2021 · 9 comments
Assignees
Labels
✨Feature This adds/improves/enhances a feature

Comments

@tamaina
Copy link
Member

tamaina commented Feb 9, 2021

Summary

MisskeyのUI的に、通知欄(notifications.vue)ではreply, quote, mentionはノートがそのまま表示されるので、ノートが表示されたらそのnotificationも既読になるのが自然

逆はやらない。通知が表示されたからと言って完全に表示されているかわからないし、画面上でnoteを完全に表示すればsubscribeが発生してnoteが既読になる。

(PR 7129でこれはやりません)

@tamaina tamaina added the ✨Feature This adds/improves/enhances a feature label Feb 9, 2021
@tamaina
Copy link
Member Author

tamaina commented Feb 14, 2021

あと、ノートが削除されたら通知が削除されるようにもしなければならない

@tamaina
Copy link
Member Author

tamaina commented Feb 14, 2021

いや、普通に消えそう

@tamaina
Copy link
Member Author

tamaina commented Feb 14, 2021

消えそうだけど、CASCADEで消えるのをフックする必要がある

@tamaina
Copy link
Member Author

tamaina commented Feb 14, 2021

あと、フォロリク/グループ招待を承認/拒否→通知既読、フォロリク/グループ招待キャンセル→通知削除もかな

@syuilo syuilo self-assigned this Jul 8, 2021
@syuilo
Copy link
Member

syuilo commented Jul 8, 2021

現在はnotifications.vueでnoteコンポーネントとnotificationコンポーネントを出しわけているけど、全部notificationコンポーネントにしてnotificationコンポーネント側で必要に応じてnoteコンポーネント呼び出したりするようにすればいい感じになるかな?

@syuilo
Copy link
Member

syuilo commented Jul 8, 2021

フォロリク/グループ招待については別途実装が必要だけど

@syuilo
Copy link
Member

syuilo commented Jul 8, 2021

ぬあーいろいろ勘違いしてた

@syuilo
Copy link
Member

syuilo commented Jul 8, 2021

普通にsearvice/note/read.tsでnotificationのread処理追加するだけで事足りそうか

@tamaina
Copy link
Member Author

tamaina commented Jul 8, 2021

ぬあーいろいろ勘違いしてた

いや、書き方が悪かった感ある(通知ではなくあなた宛てとかを例に出すべきだった

普通にsearvice/note/read.tsでnotificationのread処理追加するだけで事足りそうか

yes

@syuilo syuilo closed this as completed in 49febe1 Jul 8, 2021
snk-lab pushed a commit to snk-lab/misskey-kusa-note that referenced this issue Sep 24, 2021
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
None yet
Development

No branches or pull requests

2 participants