bootcamp
bootcamp copied to clipboard
未読通知が100件以上の場合、通知ベル上のカウントが「99+」になるようにした
Issue
- https://github.com/fjordllc/bootcamp/issues/7467
概要
画面右上の通知ベルには、未読通知数が表示されています。
以下の画像は、未読通知が200件ある状態ですが表示は「99」となってしまっています。これを「99+」という表示にしたい。
View側ではちゃんと未読通知件数が99件より多いと、表示は「99+」になる実装になっています。
https://github.com/fjordllc/bootcamp/blob/6dcc83786e9af3b2d965bd14360c382fa396e6d7/app/javascript/components/NotificationsBell/BellButton.jsx#L14-L15
原因は取得する未読通知のリミットが99となっていたためです。今回の修正箇所はここです。
https://github.com/fjordllc/bootcamp/blob/03e5c1ef175da7da65791008377da534c1ff5276/app/models/notification.rb#L47
変更確認方法
-
bug/counts_icon_of_unread_notifications
をローカルに取り込む。 - 以下のコード:
1.upto(25)
を1.upto(200)
に一時的に書き換える。 https://github.com/fjordllc/bootcamp/blob/03e5c1ef175da7da65791008377da534c1ff5276/db/fixtures/notifications.yml#L159-L160 -
rails db:seed
でseedsデータを投入する。(サーバ起動中でも実行可能です) - ユーザーhatsunoでログインをして、通知ベルの未読通知の表示を確認する。
- [ ] 通知ベルの未読通知数が「99+」になっていること
- [ ] 未読通知が1件以上99件以下の場合は、通知ベルの未読通知数がその件数「1~99」を表示していること。
- [ ] 未読通知が0件の場合は、通知ベルの未読通知数が表示されていないこと。
Screenshot
未読通知の件数が100件以上の場合
変更前
変更後
関連Issue・PR
- https://github.com/fjordllc/bootcamp/issues/7163
- https://github.com/fjordllc/bootcamp/pull/7257