Feat: 未読通知数を表示できるように

[kakkokari-gtyih]

What

• オプトインで未読通知の数を表示できるようにした
  • /api/i に unreadNotificationCount を追加
  • 100件以上の通知がある場合は「99+」という表記になります（API上では正確な数を取得可能）
• テスト通知（サーバーサイド）の動作を改善した
  • テスト通知は自動で既読がつかないように
  • テスト通知のみ、未読イベントを即時発行するように
  • テスト通知のプッシュ通知が入るように修正
• 通知画面の動作を改善した
  • 通知ページを閉じたあとは、通知ページから既読をつける動作が発火しないように

Why

Fix #11981

Additional info (optional)

デザインは改良の余地あり（文字が小さすぎる）

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e85b821) 79.18% compared to head (c9c5b23) 79.01%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #11982      +/-   ##
===========================================
- Coverage    79.18%   79.01%   -0.17%     
===========================================
  Files          944      177     -767     
  Lines       101181    23093   -78088     
  Branches      8222      493    -7729     
===========================================
- Hits         80117    18248   -61869     
+ Misses       21064     4845   -16219     

see 767 files with indirect coverage changes

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

[syuilo]

オプトインにする必要あんまり感じないかも（常にオンで良さそう）

[syuilo]

文字自体の大きさはそれで良さそうで、paddingをもうちょっと追加すればいい感じになりそう

[syuilo]

同じような(もしくは全く同じ？)スタイルが5か所くらいに追加されてるからコンポーネントにするかstyle.scssにクラスとしてまとめるのがよさそう

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

--- base
+++ head
@@ -58159,6 +58159,11 @@
             "nullable": false,
             "optional": false
           },
+          "unreadNotificationsCount": {
+            "type": "number",
+            "nullable": false,
+            "optional": false
+          },
           "mutedWords": {
             "type": "array",
             "nullable": false,
@@ -58241,6 +58246,7 @@
           "hasUnreadAntenna",
           "hasUnreadNotification",
           "hasPendingReceivedFollowRequest",
+          "unreadNotificationsCount",
           "mutedWords",
           "mutedInstances",
           "notificationRecieveConfig",

Get diff files from Workflow Page

[kakkokari-gtyih]

オプトインにする必要あんまり感じないかも（常にオンで良さそう）

デフォルトONでオフにできるようにしておきます

[kakkokari-gtyih]

[syuilo]

navbar-for-mobileとかで _indicateCounter を使っていないのは理由ある？

[syuilo]

_indicateCounter にしても問題なさそうだったから _indicateCounter にした

[syuilo]

👍🏻