bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

未読通知が5ページより前にいかない

Open komagata opened this issue 11 months ago • 6 comments

未読の通知一覧だけ5ページより前に行けない。(全ての通知のページはいける)

:memo: 未読のデータを100件以上取ってきてない動きをしているような気がする。

未読通知: image

全ての通知: image

実際に400件ぐらい未読があっても100件以上古いものにアクセスできなかった。

komagata avatar Mar 14 '24 01:03 komagata

@komagata さん お疲れ様です! 本Issueの進め方について2点お伺いさせてください🙏

  • 未読データの取得上限は外してよいでしょうか?あるいは上限数値を引き上げるべきでしょうか? (現在はkomagataさんご推測のとおり、99件を上限としているようです。また、既読も含めた通知の取得には制限がありません。)
  • Issueの再現方法はseedデータで未読通知を100件以上つくる方法でよいでしょうか?

お手すきの際にご回答いただけますと幸いです。 よろしくお願いいたします。

unikounio avatar Apr 03 '24 13:04 unikounio

@unikounio すみません、偶然Issueを見たのですが自分が担当しているものと修正スコープが同じかもしれません…!

https://github.com/fjordllc/bootcamp/blob/03e5c1ef175da7da65791008377da534c1ff5276/app/models/notification.rb#L47

https://github.com/fjordllc/bootcamp/issues/7467

お手数ですが軽く確認して頂いてもよろしいでしょうか🙏

masyuko0222 avatar Apr 04 '24 07:04 masyuko0222

@masyuko0222 さん お疲れ様です!

自分が担当しているものと修正スコープが同じかもしれません…!

同じですね😅 Discordでご提案いただいているようなので、駒形さんへのご連絡お願いできればと思います🙏 ご連絡ありがとうございます✨

unikounio avatar Apr 04 '24 08:04 unikounio

@komagata @machida @unikounio すみません、こちらのIssueとhttps://github.com/fjordllc/bootcamp/issues/7467 の修正範囲が被っていそうです。 うにおさんがすぐに実装取り掛かれそうでしたら、自分のはCloseしてうにおさんにやって頂こうかと思ってるのですが、よろしいでしょうか?

masyuko0222 avatar Apr 04 '24 08:04 masyuko0222

@masyuko0222 すいません🙇‍♂️そのようにお願いします。別のIssueをアサインします。お待ちください🙏

machida avatar Apr 04 '24 10:04 machida

@masyuko0222 もうしわけないです。

@machida 対応ありがとうございます。たすかりました。

komagata avatar Apr 07 '24 21:04 komagata

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] avatar Jun 07 '24 01:06 github-actions[bot]

本Issueは本番環境での解消が確認できたため、Closeさせていただきます。 ご協力いただいた皆様、ありがとうございました!🙏✨

unikounio avatar Jun 13 '24 10:06 unikounio