Skip to content

Delegates: server-side pagination + separate count - #88

Merged
fionnachan merged 1 commit into
mainfrom
feat/delegates-srv-pagination
Jul 30, 2026
Merged

Delegates: server-side pagination + separate count#88
fionnachan merged 1 commit into
mainfrom
feat/delegates-srv-pagination

Conversation

@douglance

Copy link
Copy Markdown
Contributor

Moves the delegates page from "load the whole list, page/sort client-side" to server-driven pagination, and fetches the eligible count from the indexer's new /api/tally/delegate-count instead of pulling the full list to count it.

  • Delegates table now fetches one page at a time (getDelegatesPage) with rowCount from the count endpoint; rank is continuous across pages.
  • Sorting preserved (Voting Power / Random / column headers) — now driven server-side via the indexer sort/dir/seed params.
  • /api/delegate-count calls the cheap count endpoint (with the governance exclude list) instead of fetching every delegate.

Needs the indexer PR (OffchainLabs/arbitrum-governance-indexer#22). Full test suite green (1119).

Move the delegates table from loading the full list + client-side paging to
server-driven pagination against the indexer's paginated list endpoint, and
fetch the eligible count from the new /api/tally/delegate-count instead of
counting a full-list fetch. Sorting (Voting Power / Random / column headers)
now drives the indexer's sort params. Exclusion is applied server-side.
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tally-zero Ready Ready Preview Jul 28, 2026 6:11pm

Request Review

@fionnachan
fionnachan merged commit 6ecb9f1 into main Jul 30, 2026
4 of 5 checks passed
@fionnachan
fionnachan deleted the feat/delegates-srv-pagination branch July 30, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants