dev → uat: GRO-2187 portal pet PATCH + GET enrichment (carries GRO-2152) #166

Merged

Flea Flicker merged 3 commits from dev-to-uat-gro-2187 into uat 2026-06-08 10:02:17 +00:00

Conversation 1 Commits 3 Files Changed 7

+575 -2

Flea Flicker commented 2026-06-08 08:19:19 +00:00

Member

Copy Link

Copy Source

Promotion: dev → uat

Promotes the current dev head to uat for QA. Two commits ride this train:

1. GRO-2187 (primary) — implement PATCH /api/portal/pets/:petId + enrich GET /api/portal/pets. Fixes the UAT §5.23 regression under GRO-1480 (portal pet-save was 404 / no persistence). Merged to dev as #165.
2. GRO-2152 (rides along) — route optimization schema migration, already merged to dev as #164. A linear dev→uat promotion necessarily carries it; flagging for visibility.

QA focus (GRO-2187)

After this deploys to UAT, re-run UAT_PLAYBOOK.md §4.8:

• TC-API-8.12 — GET /api/portal/pets returns coatType, petSizeCategory, healthAlerts, preferredCuts, medicalAlerts.
• TC-API-8.13 — owner PATCH /api/portal/pets/{petId} persists (200), web xlarge → DB extra_large; round-trip via GET.
• TC-API-8.14 — non-owner PATCH → 403 (or 404 for unknown pet).
• TC-API-8.15 — invalid coatType/petSizeCategory → 422.

This closes the API side of GRO-1480 §5.23.

CI

dev PR #165 was green on Lint & Typecheck, Test (582 passed), and Build & Push.

cc @cpfarhood

## Promotion: dev → uat Promotes the current `dev` head to `uat` for QA. Two commits ride this train: 1. **[GRO-2187](/GRO/issues/GRO-2187)** (primary) — implement `PATCH /api/portal/pets/:petId` + enrich `GET /api/portal/pets`. Fixes the UAT §5.23 regression under [GRO-1480](/GRO/issues/GRO-1480) (portal pet-save was 404 / no persistence). Merged to dev as #165. 2. **[GRO-2152](/GRO/issues/GRO-2152)** (rides along) — route optimization schema migration, already merged to dev as #164. A linear dev→uat promotion necessarily carries it; flagging for visibility. ## QA focus (GRO-2187) After this deploys to UAT, re-run **UAT_PLAYBOOK.md §4.8**: - **TC-API-8.12** — `GET /api/portal/pets` returns coatType, petSizeCategory, healthAlerts, preferredCuts, medicalAlerts. - **TC-API-8.13** — owner `PATCH /api/portal/pets/{petId}` persists (200), web `xlarge` → DB `extra_large`; round-trip via GET. - **TC-API-8.14** — non-owner PATCH → 403 (or 404 for unknown pet). - **TC-API-8.15** — invalid `coatType`/`petSizeCategory` → 422. This closes the API side of [GRO-1480](/GRO/issues/GRO-1480) §5.23. ## CI dev PR #165 was green on Lint & Typecheck, Test (582 passed), and Build & Push. cc @cpfarhood

Flea Flicker added 3 commits 2026-06-08 08:19:20 +00:00

feat(GRO-2152): route optimization schema migration ... 4884961c8e

Add the database foundation for mobile groomer route optimization:

- clients: latitude/longitude (double precision) + geocodedAt
- groomer_routes: per-(staff, date) route with route_status enum,
  totals, optimizedAt; UNIQUE(staff_id, route_date)
- route_stops: ordered stops FK->groomer_routes (cascade) + appointments,
  lat/lng, per-leg travel mins/distance, bufferMins;
  UNIQUE(route_id, appointment_id) and UNIQUE(route_id, stop_order)
- business_settings: defaultTravelBufferMins (default 15),
  routeOptimizationProvider (default nominatim), googleMapsApiKey
  (encrypted at rest at the app layer)
- Idempotent hand-authored migration 0041 + journal entry (when=max+1)

Lands in packages/db (the deployed schema/migration source per the
Dockerfile migrate stage); apps/api is the legacy CI-only copy.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Merge pull request 'feat(GRO-2152): route optimization schema migration' (#164) from feat/gro-2152-route-optimization-schema-dev into dev 40bd6dcfea

fix(portal): implement PATCH /portal/pets/:petId + enrich GET (GRO-2187) (#165) 6be78cae35

Lint Roller approved these changes 2026-06-08 08:23:57 +00:00

Lint Roller left a comment

Member

Copy Link

Copy Source

QA LGTM — all 7 unit tests pass, CI green (Lint+Typecheck, Test, Build). PATCH ownership enforcement correct (403/404), payload mapping verified (weightKg wins, xlarge→extra_large, column aliases). 422 in-handler enum validation. GET enrichment complete. UAT_PLAYBOOK §4.8 TC-API-8.12–8.15 added. Approved for uat merge.

QA LGTM — all 7 unit tests pass, CI green (Lint+Typecheck, Test, Build). PATCH ownership enforcement correct (403/404), payload mapping verified (weightKg wins, xlarge→extra_large, column aliases). 422 in-handler enum validation. GET enrichment complete. UAT_PLAYBOOK §4.8 TC-API-8.12–8.15 added. Approved for uat merge.

Flea Flicker merged commit b3db206588 into uat 2026-06-08 10:02:17 +00:00

Flea Flicker referenced this pull request 2026-06-08 10:03:14 +00:00

dev → uat: GRO-2153 abstracted geocoding service #168

Flea Flicker referenced this pull request 2026-06-08 13:14:36 +00:00

uat → main: portal pet PATCH + photoKey S3 key-hijack fix (GRO-2187) #174

Sign in to join this conversation.

Reviewers

No Reviewers

Lint Roller

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/api#166