Db pagination for tabular reviews
From the PR description
Summary
For the Tabular Reviews page, this moves the keyword search and order by functions from the data table into the backend, and adds pagination with lazy load fetching.
Why / Motivation
First part of the pagination feature request from the discord chat.
Changes
Implement offset based paging in the database for the tabular view.
Tradeoffs & risks
Considered multiple UI paging strategies and interface options, against the potential perf impacts and scope for the number of records considering this is an open source project and may be used in multi-tenanted environments.
This is an implementation on a single view, and once the approach is reviewed and approved, should be mirrored in the other views which require paging.
How verified
Added some tests, but mainly by manual testing in my local instance manually populated with seed data to test pagination.
Checklist
- Ran the relevant build/test command for the area changed.
- Reviewed
git diffand removed unrelated changes. - Updated docs / env examples if setup, config, or behavior changed.
- No secrets, API keys, real documents, or
.envfiles committed.
Our analysis
Move tabular review search and paging into the database — read the full analysis →
Think the analysis missed something the PR description covers?
Commits in this PR (21)
| SHA | Subject | Author | Date | |
|---|---|---|---|---|
0f93d8e8 | Initial pagination implementation starting with tabular reviews | Anthony May | 2026-07-25 | ↗ GitHub |
dabe4aba | Seed data for testing pagination | Anthony May | 2026-07-25 | ↗ GitHub |
aeff9fc7 | Pagination for tabular reviews smoke testing | Anthony May | 2026-07-25 | ↗ GitHub |
342702cc | Display correctly for filtered results | Anthony May | 2026-07-25 | ↗ GitHub |
cc7e399b | Wired the search into the pagination functionality | Anthony May | 2026-07-25 | ↗ GitHub |
f84c2561 | Add specific grants to fix db when restarting supabase | Anthony May | 2026-07-25 | ↗ GitHub |
7aa23595 | Search and pagination is working | Anthony May | 2026-07-26 | ↗ GitHub |
d0848d6d | Fix the keyword search, so that you can sort keyword paged results | Anthony May | 2026-07-26 | ↗ GitHub |
b854e254 | Fix pagination bug with last call when the results are exactly a page | Anthony May | 2026-07-26 | ↗ GitHub |
f08b62cc | Auto-call the load more on scroll | Anthony May | 2026-07-26 | ↗ GitHub |
614f0f09 | Move the rpc changes into a new migration file | Anthony May | 2026-07-26 | ↗ GitHub |
937d0ee2 | Fix perf issues for the pagination | Anthony May | 2026-07-26 | ↗ GitHub |
034cd6e1 | Fix perf issues for the pagination | Anthony May | 2026-07-26 | ↗ GitHub |
e76fdae7 | Merge branch 'db-pagination' of github.com:amay0048/mike-amay0048 into db-pagination | Anthony May | 2026-07-26 | ↗ GitHub |
ec325c92 | Restore the package-lock to main version | Anthony May | 2026-07-26 | ↗ GitHub |
55cc0a0c | Prevent the search box from losing focus when results are returned | Anthony May | 2026-07-26 | ↗ GitHub |
6b0bd1d4 | fix: make tabular review pagination reliable | willchen96 | 2026-07-27 | ↗ GitHub |
bf45f94b | Support select all then delete functionality with pagination | Anthony May | 2026-07-27 | ↗ GitHub |
25a082fe | Support select all then delete functionality with pagination | Anthony May | 2026-07-27 | ↗ GitHub |
ae6b797e | Merge branch 'db-pagination' of github.com:amay0048/mike-amay0048 into db-pagination | Anthony May | 2026-07-27 | ↗ GitHub |
d6b35c9a | fix: harden paginated review selection and deletion | willchen96 | 2026-07-31 | ↗ GitHub |
Capture this PR into my fork
Download a Markdown prompt that tells Claude how to port every
commit in this PR into your working tree. Run it via
claude -p < capture-pull-263.md from
inside the repo you want the changes in.