Enhance:新着ノート通知があった時まとめるように

[penginn-net]

What

新着ノート通知があった時まとめるように

Why

Resolve #328
Resolve #108

Additional info (optional)

まとめる基準は連続してノート通知があった場合連続してる分をまとめる

Checklist

• コントリビューションガイドを読みました( Read the contribution guide)
• ローカル環境で動作しました(Test working in a local environment)
• (必要なら)CHANGELOG_YOJO.mdの更新((If needed) Update CHANGELOG_YOJO.md)
• (必要なら)テストの追加((If possible) Add tests)

[github-actions]

이 PR에 의한 api.json 차이

차이점은 여기에서 볼 수 있음

--- base
+++ head
@@ -46293,6 +46293,7 @@
                         "test",
                         "reaction:grouped",
                         "renote:grouped",
+                        "note:grouped",
                         "pollVote"
                       ]
                     }
@@ -46319,6 +46320,7 @@
                         "test",
                         "reaction:grouped",
                         "renote:grouped",
+                        "note:grouped",
                         "pollVote"
                       ]
                     }

Get diff files from Workflow Page

[codecov]

Codecov Report

Attention: Patch coverage is 7.50000% with 74 lines in your changes missing coverage. Please review.

Project coverage is 40.42%. Comparing base (676dbf7) to head (edb1db6).
Report is 16 commits behind head on develop.

Files with missing lines	Patch %	Lines
...end/src/core/entities/NotificationEntityService.ts	12.50%	21 Missing ⚠️
...rc/server/api/endpoints/i/notifications-grouped.ts	0.00%	18 Missing ⚠️
...ackages/frontend/src/components/MkNotification.vue	0.00%	13 Missing ⚠️
...ckages/frontend/src/components/MkNotifications.vue	0.00%	8 Missing ⚠️
packages/frontend/src/pages/notifications.vue	0.00%	8 Missing ⚠️
packages/backend/src/models/Notification.ts	0.00%	6 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##           develop     #387       +/-   ##
============================================
+ Coverage    20.21%   40.42%   +20.21%     
============================================
  Files          786     1666      +880     
  Lines       117084   224066   +106982     
  Branches      1088     3768     +2680     
============================================
+ Hits         23665    90578    +66913     
- Misses       92831   132844    +40013     
- Partials       588      644       +56     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[penginn-net]

ノート通知は連続してなくても全部まとめもいい気もする

[penginn-net]

クリックされた時に
新着投稿一覧ページに飛ばすほうがいいかもしれない

通知の新着ノート通知だけ絞り込んで表示する感じ
≠TL

[kozakura913]

投稿者単位で束ねたい(クリックでユーザーTLに飛ぶ感じで)

[penginn-net]

ユーザーアイコンクリックで飛べばよくないかなとか

[penginn-net]

kokonectのissueに書いてある方読んでた(書いてあることが違った)
一人一人まとめるより、あっちの方が見やすそうな気がするんだけどどうかしら

[penginn-net]

間に別通知が入るとグループされなくなるのも追いづらいわね

[penginn-net]