Skip to content
Snippets Groups Projects

Add pagination logic to fetch all timeline events

Merged Warren Gifford requested to merge es/add-timeline-items-pagination into main

Created by: eseliger

Fixes values derived from events on busy PRs by using the pagination info from GraphQL to receive all pages of items. This happens when a PR has a lot of comments or a lot of commits in it, and this should ultimately fix it. I think we don't need to worry about running into rate limits, because it only happens to have so many events very rarely. (But it happened on our own repo!)

Closes #11744 (closed)

Merge request reports

Approval is optional

Merged by avatar (Jul 13, 2025 3:01am UTC)

Merge details

  • Changes merged into main with ebd5c88c.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading