feat: add limit/pagination to scout_opportunities endpoint #148
Labels
No Label
ai-changes-requested
ai-failed
ai-merged
ai-pr-opened
ai-reviewed
ai-reviewing
ai-reviewing
ai-working
bug
enhancement
evolution
performance
phase-0
phase-1a
phase-1b
phase-1c
phase-1d
security
tech-debt
todo
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: cal/paper-dynasty-database#148
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The
GET /api/v2/scout_opportunitiesendpoint returns all records with nolimitparameter. Add an optionallimit: intquery param (default 100, max 500).Found during smoke test development — endpoints without limits cause excessive data transfer.
PR #154 opened: #154
Added
limit: Optional[int] = 100toGET /api/v2/scout_opportunities, capped at 500 viamax(0, min(limit, 500)). Also removed a pre-existing unusedfnimport (required to pass ruff pre-commit hook).