Skip to content

Conversation

@doneill
Copy link
Owner

@doneill doneill commented Aug 17, 2025

Add patrol segment id and end time to patrols command output

Proposed Changes

  • Displays both start and end location
  • Displays the patrols segment id

Expectation:

  • Patrol id and end location included in patrols command output
# display patrols for last 7 days and status
./er patrols -d 7
|--------|-------|--------------------------------------|--------------------------------------|------------------------|-----------------|------------------------|------------------------|--------------|--------------|
| SERIAL | STATE |                  ID                  |              SEGMENT ID              |         TITLE          |     LEADER      |     START LOCATION     |      END LOCATION      |  START TIME  |   END TIME   |
|--------|-------|--------------------------------------|--------------------------------------|------------------------|-----------------|------------------------|------------------------|--------------|--------------|
  • Patrols test pass
=== RUN   TestPatrols
=== RUN   TestPatrols/successful_response_without_filters
=== RUN   TestPatrols/successful_response_with_done_patrol_having_end_location_and_time
=== RUN   TestPatrols/successful_response_with_status_filter
=== RUN   TestPatrols/error_response
--- PASS: TestPatrols (0.00s)
    --- PASS: TestPatrols/successful_response_without_filters (0.00s)
    --- PASS: TestPatrols/successful_response_with_done_patrol_having_end_location_and_time (0.00s)
    --- PASS: TestPatrols/successful_response_with_status_filter (0.00s)
    --- PASS: TestPatrols/error_response (0.00s)

@doneill doneill self-assigned this Aug 17, 2025
@doneill doneill merged commit b841446 into develop Aug 17, 2025
1 check passed
@doneill doneill deleted the task/jdo-43-patrol-segments branch August 17, 2025 20:31
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.

2 participants