未読通知を100件以上取得できるようにした

[unikounio]

Issue

• 未読通知が５ページより前にいかない #7539

概要

99件までとされていた未読通知のデータ取得制限を取り払いました。
これにより、通知数が101件を超えても5ページまでしか表示されなかった未読通知一覧ページが、6ページ以上表示されるようになります。

変更確認方法

1. bug/unread-notifications-only-5-pagesをローカルに取り込む
2. bin/setupを実行
3. foreman start -f Procfile.devでサーバーを立ち上げる
4. ユーザー名hatsunoでログイン
5. 画面右上の通知ベルの表示が99+となっていることを確認し、プルダウンメニューから「全ての未読通知一覧へ」をクリック
   • 通知数が少ない場合は、bin/rails db:seedを実行してください
6. 未読通知一覧ページが6ページ以上あることを確認する

Screenshot

いずれも未読通知数100件以上で撮影しています。

変更前

変更後

[unikounio]

@a-terumoto-gs さん
お疲れ様です！
こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏
ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。
よろしくお願いいたします。

[a-terumoto-gs]

@unikounio
お疲れ様です。
フィヨルド以外の業務が忙しく、確認できるのがだいぶ先になってしまうかもしれません(;´･ω･)
他の方にお願いしてもらえると助かりますm(__)m
申し訳ございませんがよろしくお願いいたします。

[unikounio]

@a-terumoto-gs さん
ご連絡ありがとうございます！
お忙しいところ失礼いたしました🙏
別の方に依頼させていただきます～

[unikounio]

@dowdiness さん
お疲れ様です！
こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏
急ぎではありませんので、ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。
よろしくお願いいたします。

お忙しいようでしたので別の方に依頼させていただきます！
失礼いたしました🙏

[unikounio]

@ham-cap さん
お疲れさまです！
こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏
急ぎではありませんので、ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。
よろしくお願いいたします。

[ham-cap]

@unikounio
お疲れ様です！レビューにつきまして承知いたしました👍
本日中は難しいのですが、明日以降2〜3日中に確認させていただきます🙏

[ham-cap]

@unikounio
お疲れ様です！
99件までの制限が無くなり、未読が101件以上ある場合には一覧ページも6ページ以降に続いていることを確認いたしました👍
コードの修正箇所についても私としては特に気になる点等なかったので、これでApproveさせていただきますー🎉

[unikounio]

@ham-cap さん
ご確認ありがとうございます！
助かりました～😄

[unikounio]

@komagata さん
お疲れ様です！
チームメンバーよりApproveをいただきましたので、お手すきの際にレビューをお願いいたします🙏

[komagata]

conflictの修正をお願い致します～。

[unikounio]

@komagata さん
失礼いたしました！
conflictを修正させていただきましたので、お手すきの際に再度ご確認をお願いいたします🙏

[komagata]

確認させていただきました。OKです～👌

[unikounio]

@komagata さん、 @machida さん
お疲れ様です！
こちらのPRについて、本番環境でのご確認をお願いできないでしょうか🙏

メンター・管理者権限が必要なPRではないのですが、確認のため未読通知が100件以上必要なので、メンター・管理者の方々の方が確認条件を満たされている可能性が高いと考えお願いさせていただきました。
難しいようでしたらスクラムメンバーに協力を依頼して、一時的にテスト用の日報を100件つくって確認しようと思います。

よろしくお願いいたしますmm

[machida]

@unikounio 確認できましたー🙆

[unikounio]

@machida さん
ご対応ありがとうございます！IssueをCloseさせていただきます～