Skip to content

Police Pagination#297

Open
VidurShah wants to merge 1 commit intomainfrom
vidur/police-pagination
Open

Police Pagination#297
VidurShah wants to merge 1 commit intomainfrom
vidur/police-pagination

Conversation

@VidurShah
Copy link
Copy Markdown
Collaborator

@VidurShah VidurShah commented Apr 7, 2026

Wide date ranges on the police view could render hundreds of party cards at once, overloading the browser.

Changes:

  • Added pagination to the police party list (10 entries per page), mirroring the shadcn Pagination controls used on the staff view
  • Clicking a map pin still navigates to the correct page and scrolls the card into view
  • Fixed a 422 caused by page_size: 200 exceeding the backend max — police query now omits page_size to fetch all parties in the date range (required for client-side filters to work across the full result set)
  • Fixed party_datetime_lte being sent as midnight instead of end of day, which caused the default single-day filter to return no results

Closes #277

@VidurShah VidurShah marked this pull request as draft April 7, 2026 21:38
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

Test Results Summary

551 tests  ±0   551 ✅ ±0   38s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 2a77c1c. ± Comparison against base commit 46dbc32.

@VidurShah VidurShah marked this pull request as ready for review April 7, 2026 21:40
@VidurShah VidurShah requested a review from naasanov April 7, 2026 21:40
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.

Police Pagination

1 participant