Fix cursor condition grouping for pagination #1714

Merged
mfreeman451 merged 1 commit from refs/pull/1714/head into 898-featarmis-north-bound-integration-with-sweep-results-to-armis 2025-06-16 20:16:50 +00:00
mfreeman451 commented 2025-06-16 20:16:06 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #935
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/935
Original created: 2025-06-16T20:16:06Z
Original updated: 2025-06-23T21:00:17Z
Original head: carverauto/serviceradar:codex/write-tests-for-query-failure-issue
Original base: 898-featarmis-north-bound-integration-with-sweep-results-to-armis
Original merged: 2025-06-16T20:16:50Z by @mfreeman451

Summary

  • wrap cursor conditions in a complex clause so pagination queries are valid
  • update tests for new cursor condition structure

Testing

  • go test ./... -count=1

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

Imported from GitHub pull request. Original GitHub pull request: #935 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/935 Original created: 2025-06-16T20:16:06Z Original updated: 2025-06-23T21:00:17Z Original head: carverauto/serviceradar:codex/write-tests-for-query-failure-issue Original base: 898-featarmis-north-bound-integration-with-sweep-results-to-armis Original merged: 2025-06-16T20:16:50Z by @mfreeman451 --- ## Summary - wrap cursor conditions in a complex clause so pagination queries are valid - update tests for new cursor condition structure ## Testing - `go test ./... -count=1` ------ https://chatgpt.com/codex/tasks/task_e_685072620fe883209bd4872fad4b6df2
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!1714
No description provided.