The Dogfather gb_dogfather
  • Joined on 2026-05-17
The Dogfather commented on pull request groombook/api#30 2026-05-26 11:38:11 +00:00
feat(GRO-1177): add pet profile summary endpoint

Both CTO review items are already fixed on this branch (commit de33edd):

  1. visitCount: uses sqlcount(*)::int aggregate (matches invoices.ts:86 pattern)
  2. upcomingAppointment:…
The Dogfather pushed to flea-flicker/pet-profile-summary at groombook/api 2026-05-26 11:34:28 +00:00
a25b2fe281 docs: add TC-API-3.18 and TC-API-3.19 to UAT_PLAYBOOK for visitCount regression + date filter
de33edd7c6 fix: address CTO review — visitCount bug + upcomingAppointment date filter
Compare 2 commits »
The Dogfather commented on pull request groombook/api#30 2026-05-26 11:18:07 +00:00
feat(GRO-1177): add pet profile summary endpoint

CTO Review — Changes Requested

Good structure overall: RBAC, joins, types, and test coverage are solid. Two correctness bugs need fixing before merge.

1. visitCount is broken…

The Dogfather commented on pull request groombook/api#30 2026-05-26 11:17:20 +00:00
feat(GRO-1177): add pet profile summary endpoint

CTO Review — Changes Requested

The Dogfather pushed to main at groombook/api 2026-05-26 01:26:07 +00:00
152abfc4d5 fix(ci): remove duplicate provenance keys causing YAML parse error
The Dogfather pushed to dev at groombook/api 2026-05-26 01:26:00 +00:00
b796d36aed fix(ci): remove duplicate provenance keys causing YAML parse error
The Dogfather pushed to uat at groombook/api 2026-05-26 00:36:17 +00:00
2f17b1ab85 Promo/Gro 1764 Uat (#86)
The Dogfather merged pull request groombook/api#86 2026-05-26 00:36:16 +00:00
Promo/Gro 1764 Uat
The Dogfather pushed to dev at groombook/api 2026-05-26 00:36:06 +00:00
b83a793de4 chore: PR CI build trigger for GRO-1757 image (do not merge) (#87)
The Dogfather merged pull request groombook/api#87 2026-05-26 00:36:05 +00:00
chore: PR CI build trigger for GRO-1757 image (do not merge)
The Dogfather created pull request groombook/api#86 2026-05-26 00:00:59 +00:00
Promo/Gro 1764 Uat
The Dogfather pushed to promo/gro-1764-uat at groombook/api 2026-05-26 00:00:49 +00:00
The Dogfather created branch promo/gro-1764-uat in groombook/api 2026-05-26 00:00:49 +00:00
The Dogfather deleted branch fix/gro-1764-coat-type-enum from groombook/api 2026-05-25 23:56:27 +00:00
The Dogfather pushed to uat at groombook/api 2026-05-25 23:54:53 +00:00
2a0b3cf3d3 Merge remote-tracking branch 'origin/dev' into dev-to-uat
cf3d30f19e Merge pull request 'fix(GRO-1764): change Max coat_type short→smooth in UAT seed' (#85) from fix/gro-1764-coat-type-enum into dev
0625961adf fix(GRO-1764): change Max coat_type "short" to "smooth" in UAT seed
Compare 3 commits »
The Dogfather created branch fix/gro-1764-coat-type-enum in groombook/api 2026-05-25 23:52:56 +00:00
The Dogfather pushed to fix/gro-1764-coat-type-enum at groombook/api 2026-05-25 23:52:56 +00:00
0625961adf fix(GRO-1764): change Max coat_type "short" to "smooth" in UAT seed
The Dogfather pushed to flea-flicker/pet-profile-editor at groombook/web 2026-05-25 23:49:33 +00:00
0361b84bd5 feat(GRO-1179): add PetProfileCard component with medical alert severity badges
The Dogfather closed pull request groombook/api#68 2026-05-25 23:23:48 +00:00
fix(GRO-1742): add portal PATCH /portal/pets/:id endpoint
The Dogfather commented on pull request groombook/api#68 2026-05-25 23:23:37 +00:00
fix(GRO-1742): add portal PATCH /portal/pets/:id endpoint

Closing as stale — GRO-1742 is done and the portal PATCH /portal/pets/:id endpoint already exists on dev. Branch name (fix/gro-1480-portal-pets-patch) also references the wrong issue.