feat: add limit/pagination to rewards endpoint #139

Closed
opened 2026-03-24 03:39:57 +00:00 by cal · 1 comment
Owner

The GET /api/v2/rewards endpoint returns all 8,800+ records with no limit parameter. Add an optional limit: int query param (default 100, max 500).

Found during smoke test development — this endpoint returns the entire table (~9k rows) causing slow responses.

The `GET /api/v2/rewards` endpoint returns all 8,800+ records with no `limit` parameter. Add an optional `limit: int` query param (default 100, max 500). Found during smoke test development — this endpoint returns the entire table (~9k rows) causing slow responses.
Claude added the
ai-working
label 2026-03-24 05:01:04 +00:00
Claude removed the
ai-working
label 2026-03-24 05:02:25 +00:00
Collaborator

PR opened: #152

Added limit: Optional[int] = 100 to GET /api/v2/rewards, capped at 500 via max(0, min(limit, 500)). Applied after all filters.

PR opened: https://git.manticorum.com/cal/paper-dynasty-database/pulls/152 Added `limit: Optional[int] = 100` to `GET /api/v2/rewards`, capped at 500 via `max(0, min(limit, 500))`. Applied after all filters.
Claude added the
ai-pr-opened
label 2026-03-24 05:02:29 +00:00
cal closed this issue 2026-03-24 12:09:59 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-database#139
No description provided.