未読通知が100件以上の場合、通知ベル上のカウントが「99+」になるようにした

[masyuko0222]

Issue

• 通知ベル上の未読通知数のカウントが、未読通知が「100件以上」存在している場合でも「99」になってしまう #7467

概要

画面右上の通知ベルには、未読通知数が表示されています。
以下の画像は、未読通知が２００件ある状態ですが表示は「99」となってしまっています。これを「99+」という表示にしたい。

View側ではちゃんと未読通知件数が99件より多いと、表示は「99+」になる実装になっています。

bootcamp/app/javascript/components/NotificationsBell/BellButton.jsx

Lines 14 to 15 in 6dcc837

	const count = notifications.length
	return count > 99 ? '99+' : String(count)

原因は取得する未読通知のリミットが99となっていたためです。今回の修正箇所はここです。

bootcamp/app/models/notification.rb

Line 47 in 03e5c1e

scope :by_read_status, ->(status) { status == 'unread' ? unreads.with_avatar.limit(99) : with_avatar }

変更確認方法

1. bug/counts_icon_of_unread_notificationsをローカルに取り込む。
2. 以下のコード：1.upto(25)を1.upto(200)に一時的に書き換える。
   

   bootcamp/db/fixtures/notifications.yml

   Lines 159 to 160 in 03e5c1e

   	<% 1.upto(25) do |i| %>
   	notification_for_pagination<%= i %>: # 通知ページのページネーション動作確認のための通知

3. rails db:seedでseedsデータを投入する。（サーバ起動中でも実行可能です）
4. ユーザーhatsunoでログインをして、通知ベルの未読通知の表示を確認する。

• 通知ベルの未読通知数が「99+」になっていること
• 未読通知が1件以上99件以下の場合は、通知ベルの未読通知数がその件数「1~99」を表示していること。
• 未読通知が0件の場合は、通知ベルの未読通知数が表示されていないこと。

Screenshot

未読通知の件数が100件以上の場合

変更前

変更後

関連Issue・PR

• 通知ベルをクリックして表示される通知一覧に、未読・既読のタブを付けたい。 #7163
• 通知ベルをクリックして表示される通知一覧に、未読・既読のタブを付けた #7257