Fix count query handling #1878

Merged
mfreeman451 merged 1 commit from refs/pull/1878/head into 1126-featui-add-sidebar-navigation 2025-06-25 01:19:27 +00:00
mfreeman451 commented 2025-06-25 01:18:57 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #1133
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/1133
Original created: 2025-06-25T01:18:57Z
Original updated: 2025-06-25T02:17:45Z
Original head: carverauto/serviceradar:codex/fix-count-devices-query-error
Original base: 1126-featui-add-sidebar-navigation
Original merged: 2025-06-25T01:19:27Z by @mfreeman451

Summary

  • avoid setting default pagination for COUNT queries
  • test pagination skipping for count queries

Testing

  • go test ./...
  • npm run lint (fails: next not found)

https://chatgpt.com/codex/tasks/task_e_685b47fff03883208bf5b6f5811b42e4

Imported from GitHub pull request. Original GitHub pull request: #1133 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/1133 Original created: 2025-06-25T01:18:57Z Original updated: 2025-06-25T02:17:45Z Original head: carverauto/serviceradar:codex/fix-count-devices-query-error Original base: 1126-featui-add-sidebar-navigation Original merged: 2025-06-25T01:19:27Z by @mfreeman451 --- ## Summary - avoid setting default pagination for `COUNT` queries - test pagination skipping for count queries ## Testing - `go test ./...` - `npm run lint` *(fails: `next` not found)* ------ https://chatgpt.com/codex/tasks/task_e_685b47fff03883208bf5b6f5811b42e4
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
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
carverauto/serviceradar!1878
No description provided.