bootcamp
bootcamp copied to clipboard
未読通知が5ページより前にいかない
未読の通知一覧だけ5ページより前に行けない。(全ての通知のページはいける)
:memo: 未読のデータを100件以上取ってきてない動きをしているような気がする。
未読通知:
全ての通知:
実際に400件ぐらい未読があっても100件以上古いものにアクセスできなかった。
@komagata さん お疲れ様です! 本Issueの進め方について2点お伺いさせてください🙏
- 未読データの取得上限は外してよいでしょうか?あるいは上限数値を引き上げるべきでしょうか? (現在はkomagataさんご推測のとおり、99件を上限としているようです。また、既読も含めた通知の取得には制限がありません。)
- Issueの再現方法はseedデータで未読通知を100件以上つくる方法でよいでしょうか?
お手すきの際にご回答いただけますと幸いです。 よろしくお願いいたします。
@unikounio すみません、偶然Issueを見たのですが自分が担当しているものと修正スコープが同じかもしれません…!
https://github.com/fjordllc/bootcamp/blob/03e5c1ef175da7da65791008377da534c1ff5276/app/models/notification.rb#L47
https://github.com/fjordllc/bootcamp/issues/7467
お手数ですが軽く確認して頂いてもよろしいでしょうか🙏
@masyuko0222 さん お疲れ様です!
自分が担当しているものと修正スコープが同じかもしれません…!
同じですね😅 Discordでご提案いただいているようなので、駒形さんへのご連絡お願いできればと思います🙏 ご連絡ありがとうございます✨
@komagata @machida @unikounio すみません、こちらのIssueとhttps://github.com/fjordllc/bootcamp/issues/7467 の修正範囲が被っていそうです。 うにおさんがすぐに実装取り掛かれそうでしたら、自分のはCloseしてうにおさんにやって頂こうかと思ってるのですが、よろしいでしょうか?
@masyuko0222 すいません🙇♂️そのようにお願いします。別のIssueをアサインします。お待ちください🙏
@masyuko0222 もうしわけないです。
@machida 対応ありがとうございます。たすかりました。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
本Issueは本番環境での解消が確認できたため、Closeさせていただきます。 ご協力いただいた皆様、ありがとうございました!🙏✨