Promote: dev → uat (GRO-1945 root src/pets visit-count hotfix) #108

Merged

The Dogfather merged 3 commits from dev into uat 2026-05-30 03:45:39 +00:00

Conversation 1 Commits 3 Files Changed 2

+98 -98

Scrubs McBarkley commented 2026-05-30 03:28:51 +00:00

Owner

Copy Link

Copy Source

Summary

Promotes the GRO-1945 hotfix (repair root src/routes/pets.ts visit-count query) from dev into uat, fixing CI run #2099 build failure.

References

• GRO-1935 (original promotion)
• GRO-1945 (hotfix: repair root src/routes/pets.ts visit-count query)
• Original broken PR: groombook/api#105

Commits included

• a14bb5e17d fix(api): repair root src/routes/pets.ts visit-count query (GRO-1945)
• 981a257d2d Merge pull request #107 from flea/GRO-1945-pets-visitcount-hotfix into dev

CI

Verify Build & Push Docker Images turns green.

cc @cpfarhood

## Summary Promotes the GRO-1945 hotfix (repair root src/routes/pets.ts visit-count query) from dev into uat, fixing CI run #2099 build failure. ## References - GRO-1935 (original promotion) - GRO-1945 (hotfix: repair root src/routes/pets.ts visit-count query) - Original broken PR: groombook/api#105 ## Commits included - `a14bb5e17d` fix(api): repair root src/routes/pets.ts visit-count query (GRO-1945) - `981a257d2d` Merge pull request #107 from flea/GRO-1945-pets-visitcount-hotfix into dev ## CI Verify Build & Push Docker Images turns green. cc @cpfarhood

Scrubs McBarkley added 2 commits 2026-05-30 03:28:51 +00:00

fix(api): repair root src/routes/pets.ts visit-count query (GRO-1945) ... a14bb5e17d

Use sql\`count(*)::int\` instead of selecting appointments.id, which was
causing TS2339 under noUncheckedIndexedAccess (arr[0] is T | undefined).

Import sql from @groombook/db. Use countRow?.count ?? 0 to stay
noUncheckedIndexedAccess-safe.

Matches the working implementation in apps/api/src/routes/pets.ts:365.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'fix(api): repair root src/routes/pets.ts visit-count query (GRO-1945)' (#107) from flea/GRO-1945-pets-visitcount-hotfix into dev 981a257d2d

Lint Roller approved these changes 2026-05-30 03:37:37 +00:00

Lint Roller left a comment

Member

Copy Link

Copy Source

LGTM — CI all green, code fix correct, UAT playbook covers TC-API-3.18.

LGTM — CI all green, code fix correct, UAT playbook covers TC-API-3.18.

Flea Flicker added 1 commit 2026-05-30 03:42:46 +00:00

GRO-1921: Fix UAT reset CronJob to seed full UAT profile with extended pet fields (#106) 0ab16b82e0

The Dogfather merged commit a5bd9c915c into uat 2026-05-30 03:45:39 +00:00

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/api#108