cal/major-domo-database
1
0
Fork 0
Code Issues 6 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

feat: increase MAX_LIMIT to 1000 for /plays/fielding/, /plays/batting/, /plays/pitching/ #111

New Issue
Closed
opened 2026-04-08 05:05:29 +00:00 by cal · 1 comment
cal commented 2026-04-08 05:05:29 +00:00
Owner
Copy Link

Increase the maximum allowed limit query parameter from the current DEFAULT_LIMIT/MAX_LIMIT to 1000 for the play-level stat endpoints: /api/v3/plays/fielding/, /api/v3/plays/batting/, /api/v3/plays/pitching/. These endpoints return individual play records which often need higher pagination limits for full-season queries. Other endpoints should keep their current limits.

Increase the maximum allowed limit query parameter from the current DEFAULT_LIMIT/MAX_LIMIT to 1000 for the play-level stat endpoints: /api/v3/plays/fielding/, /api/v3/plays/batting/, /api/v3/plays/pitching/. These endpoints return individual play records which often need higher pagination limits for full-season queries. Other endpoints should keep their current limits.
Claude added the
ai-working
 label 2026-04-08 11:31:42 +00:00
Claude removed the
ai-working
 label 2026-04-08 11:33:05 +00:00
Claude commented 2026-04-08 11:33:07 +00:00
Collaborator
Copy Link

PR #113 opened: #113

Changed le=MAX_LIMIT (500) to le=1000 in the limit parameter for /plays/batting, /plays/fielding, and /plays/pitching. Also removed the now-unused MAX_LIMIT import from each file. All other endpoints are unchanged.

PR #113 opened: https://git.manticorum.com/cal/major-domo-database/pulls/113 Changed `le=MAX_LIMIT` (500) to `le=1000` in the `limit` parameter for `/plays/batting`, `/plays/fielding`, and `/plays/pitching`. Also removed the now-unused `MAX_LIMIT` import from each file. All other endpoints are unchanged.
Claude added the
ai-pr-opened
 label 2026-04-08 11:33:13 +00:00
cal closed this issue 2026-04-08 12:53:41 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
ai-changes-requested

   
ai-failed

   
ai-pr-opened

   
ai-reviewed

   
ai-reviewing

   
ai-working
No Label
ai-changes-requested
ai-failed
ai-pr-opened
ai-reviewed
ai-reviewing
ai-working
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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/major-domo-database#111
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?