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.

Sign up for GitHub

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

Jump to bottom

お知らせ機能の強化 #11328

Closed
syuilo opened this issue Jul 20, 2023 · 21 comments · Fixed by #11497
Closed

お知らせ機能の強化 #11328

syuilo opened this issue Jul 20, 2023 · 21 comments · Fixed by #11497
Assignees
@syuilo
Labels
✨Feature This adds/improves/enhances a feature 🔥high priority

Comments

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

全体としてのお知らせではなく、ユーザー個別に運営 or システムからお知らせを送りたい時がある

ユースケース

  • 規約違反した際の警告
  • データのインポート/エクスポートジョブが完了/失敗したときの通知

実装案

  • 「お知らせ」に「あなた宛て」タブを作る？
  • 通知の種類にお知らせを追加する？
  • その他？

Related #7151
@syuilo syuilo added ✨Feature This adds/improves/enhances a feature 🔥high priority labels Jul 20, 2023
@syuilo
Copy link
Member Author

syuilo commented Jul 20, 2023

規約違反した際の警告とか、必ず読んでほしいお知らせの場合、Misskey Webアクセス時にダイアログで強制的に表示するとかもしたいから、そうすると「通知の種類として『お知らせ』を新設する」はちょっと違うかも

@syuilo
Copy link
Member Author

syuilo commented Jul 20, 2023

そのお知らせに対してユーザーが返信したりできるようにするかによっても実装が変わる

@syuilo
Copy link
Member Author

syuilo commented Jul 20, 2023

返信までできるようにすると急激に実装コストが跳ね上がるから多分やらない

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

そういうロールのユーザーからダイレクトノートが届くと目立つように表示されるみたいな感じでいいのでは

@syuilo
Copy link
Member Author

syuilo commented Jul 20, 2023

送信元ユーザーの存在が必要なのは微妙

@YTJVDCM
Copy link

YTJVDCM commented Jul 20, 2023

既存の通知カラムに表示 + クライアント起動時に画面全体に表示(アップデート時の表示みたいな感じで)とかはどうでしょう?
通知内容に重要フラグをつけて、重要な内容であれば画面全体表示や一定期間は常時最上部に表示を行うとかでも

@syuilo
Copy link
Member Author

syuilo commented Jul 21, 2023

通知は揮発性がある設計だから見逃して困るような重要な情報は置きたくないというのがある

@syuilo
Copy link
Member Author

syuilo commented Jul 21, 2023

announcementに宛先ユーザーと表示の仕方を設定できるようにするのが一番良さそう

@syuilo
Copy link
Member Author

syuilo commented Jul 21, 2023 •
edited
Loading

表示の方法

  • お知らせページ掲載(今まで通り)
  • お知らせページ掲載 + バナー表示
    • メンテナンスのお知らせとか
  • お知らせページ掲載 + ダイアログ表示
    • 何らかの警告とか

@syuilo syuilo self-assigned this Jul 21, 2023
@syuilo syuilo changed the title ユーザー個別のお知らせ お知らせ機能の強化 Jul 21, 2023
@syuilo
Copy link
Member Author

syuilo commented Jul 21, 2023 •
edited
Loading

あとバナー表示とかにする場合、新規ユーザー/長期間使っていなかったユーザーだと過去のお知らせの表示で埋め尽くされかねないから、お知らせの期限を設定できるようにしたり、登録以前のお知らせは既読扱いにするようにしたい

@syuilo
Copy link
Member Author

syuilo commented Jul 21, 2023

期限を設定できるようにするのもそうだし、管理者が手動でお知らせを済んだものとしてマークできるようにしたい(削除だと後から見直せないのが不便そう)

@syuilo
Copy link
Member Author

syuilo commented Jul 21, 2023

あとメールで送るかどうかも設定できるようにしてもいいかもしれないけど、読んだかどうかを判定できないからMisskey開いたときにすでに見たお知らせが再度表示されることになるのは微妙

@tamaina tamaina mentioned this issue Jul 29, 2023
Open
@syuilo
Copy link
Member Author

syuilo commented Aug 8, 2023

やるか

@syuilo
Copy link
Member Author

syuilo commented Aug 9, 2023

バナーとかダイアログで表示することを考えるとMisskey Web読み込み時に何らかの方法でannouncementsを取得する必要があるな

@syuilo
Copy link
Member Author

syuilo commented Aug 9, 2023

metaで良いかしら

@syuilo
Copy link
Member Author

syuilo commented Aug 9, 2023

metaに入れちゃうとキャッシュできなくなるな(announcementはユーザーによって異なる可能性があるため)

@syuilo
Copy link
Member Author

syuilo commented Aug 9, 2023

あんまり読み込み時のリクエスト増やしたくないけどapi/announcements叩かせるしかないか

@syuilo
Copy link
Member Author

syuilo commented Aug 9, 2023

効率のみ考える場合 api/i に入れるのが良さそうだけど不自然

@syuilo
Copy link
Member Author

syuilo commented Aug 9, 2023

既にhasUnreadAnnouncements返してるしそこまで不自然でもないか

@syuilo syuilo mentioned this issue Aug 9, 2023
Merged
9 tasks
@Sayamame-beans
Copy link
Member

過去に送信済みのお知らせについて、管理者側が表示順を並び替えられる機能もあると嬉しいかなと思うのですが、いかがでしょうか?

通知は揮発性がある設計だから見逃して困るような重要な情報は置きたくないというのがある

というコメントから、お知らせとして置くべきものでは無いのかもしれませんが、重要度が比較的低いお知らせが上の方に残ってしまい、後から来た方が気付きづらいことがあります。
"重要なお知らせ"という区分が出来たとしても、その中の古い物の方が重要度が高いようなケースで、見逃されやすい可能性があるかもしれません…?

@syuilo
Copy link
Member Author

syuilo commented Aug 10, 2023

効率のみ考える場合 api/i に入れるのが良さそうだけど不自然

iに入れると非ログイン時にお知らせ表示できないわね

@shiosyakeyakini-info shiosyakeyakini-info mentioned this issue Sep 7, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@syuilo syuilo

Labels
✨Feature This adds/improves/enhances a feature 🔥high priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: refine announcement misskey-dev/misskey
4 participants
@syuilo @tamaina @YTJVDCM @Sayamame-beans