Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

お知らせのページネーションが機能しない #11800

Closed
shiosyakeyakini-info opened this issue Sep 7, 2023 · 0 comments · Fixed by #11890
Closed

お知らせのページネーションが機能しない #11800

shiosyakeyakini-info opened this issue Sep 7, 2023 · 0 comments · Fixed by #11890
Labels
⚠️bug? This might be a bug

Comments

@shiosyakeyakini-info
Copy link
Contributor

💡 Summary

#11328 に関連して、お知らせのページネーションが機能しなくなっています。
api/announcementsにおいてuntilIdが無視されており、必ず一番上からlimitで指定した数までしか取得できません。

なお、Misskey.ioは untilIdとsinceIdをパラメータから削除し、かわりにoffsetで取得するように変更されているようです。

🥰 Expected Behavior

30個以上のお知らせがある場合、「もっと見る」を押下すると31個目以降のお知らせが表示される。

🤬 Actual Behavior

30個以上のお知らせがある場合でも、「お知らせ」の「もっと見る」を押下しても続きを取得することができず、APIリクエスト完了後「もっと見る」の表示が出続けてしまう。

📝 Steps to Reproduce

  1. 管理者がコントロールパネルで31個以上のお知らせを登録する(すべて未アーカイブ、またはアーカイブ状態とする)
  2. お知らせ画面を開く
  3. 最初の10個が表示されることを確認し、スクロール後続いて30個が表示されることを確認する
  4. 「もっと見る」を押下する。

📌 Environment

💻 Frontend

  • Model and OS of the device(s):
    Windows 11
  • Browser: Chrome 116.0.5845.179
  • Server URL: stg.miria.shiosyakeyakini.info
  • Misskey:
    stg.miria.shiosyakeyakini.info: 2023.9.0-beta.4

🛰 Backend (for server admin)

  • Installation Method or Hosting Service: Misskey install shell script
  • Misskey: 2023.9.0-beta.4
  • Node: v20.5.1
  • PostgreSQL: psql (PostgreSQL) 15.4 (Ubuntu 15.4-0ubuntu0.23.04.1)
  • Redis: Redis server v=7.0.11 sha=00000000:0 malloc=jemalloc-5.2.1 bits=64 build=3af367a78d5e21e9
  • OS and Architecture: Ubuntu 23.04 (GNU/Linux 6.2.0-25-generic x86_64)>
@shiosyakeyakini-info shiosyakeyakini-info added the ⚠️bug? This might be a bug label Sep 7, 2023
syuilo pushed a commit that referenced this issue Sep 24, 2023
* fix(backend): お知らせのページネーションが機能しない #11800

* Update CHANGELOG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant