promote: dev → uat (GRO-1173 buffer rules + GRO-1470 pet save persistence) #14

Merged

The Dogfather merged 14 commits from dev into uat 2026-05-21 19:46:41 +00:00

Conversation 0 Commits 14 Files Changed 21

+471 -106

The Dogfather commented 2026-05-21 19:46:34 +00:00

Member

Copy Link

Copy Source

Promotion: dev → uat

Source PRs:

• web/#13 — feat(GRO-1173): admin UI buffer rules, service default buffer, pet size/coat
• Includes GRO-1470 fix: pet profile save now persists via API

Changes:

• Buffer rules CRUD in Settings page
• Service default buffer minutes field
• Pet size category and coat type dropdowns (admin + portal)
• Pet profile save now calls PATCH API (was local-state-only)
• CI migrated to Gitea Container Registry
• UAT Playbook updated with new test cases

Note: UAT login is currently blocked pending [GRO-1478] Authentik secret re-seal. This promotion deploys code; UAT regression will run after credentials are fixed.

Merged-By: The Dogfather (CTO)

## Promotion: dev → uat **Source PRs:** - web/#13 — feat(GRO-1173): admin UI buffer rules, service default buffer, pet size/coat - Includes GRO-1470 fix: pet profile save now persists via API **Changes:** - Buffer rules CRUD in Settings page - Service default buffer minutes field - Pet size category and coat type dropdowns (admin + portal) - Pet profile save now calls PATCH API (was local-state-only) - CI migrated to Gitea Container Registry - UAT Playbook updated with new test cases **Note:** UAT login is currently blocked pending [GRO-1478] Authentik secret re-seal. This promotion deploys code; UAT regression will run after credentials are fixed. Merged-By: The Dogfather (CTO)

The Dogfather added 14 commits 2026-05-21 19:46:35 +00:00

fix: remove unused X import from lucide-react ... d1f8d27d1c

Resolves ESLint error: 'X' is defined but never used
GRO-1347

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

fix(GRO-1361): remove unused X import and delete corrupted demo-pet images ... 465db89ab4

- Remove unused 'X' import from lucide-react in PetProfiles.tsx
- Delete 10 corrupted demo-pet PNG files that contain Alibaba AccessDenied XML

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

fix(GRO-1366): add non-null assertion to removeButtons[0] ... f414d2589f

Fix TypeScript error on line 114: HTMLElement | undefined is not
assignable to Element. Added ! assertion since length guard already
excludes the empty-array case.

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

fix(GRO-903): resolve CI/CD blockers on groombook/web PR #1 ... 42f3e3211a

- Move CI workflow from .github/workflows/ to .gitea/workflows/
- Add uat branch to CI triggers (push and pull_request)
- Fix Dockerfile HEALTHCHECK to use wget instead of curl

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

Merge pull request 'feat(GRO-1174): add pet size/coat dropdowns to booking wizard' (#8) from flea-flicker/pet-profile-editor into dev ... 56b11befe9

test merge

feat(GRO-1173): apply buffer rules UI changes to extracted groombook/web repo ... ec17f1e885

This commit ports the GRO-1173 admin UI changes from the app monorepo
into the extracted groombook/web repo, using the correct source paths
(src/ instead of apps/web/src/):

- New BufferRulesSection component (full CRUD UI for /api/buffer-rules)
- Default Buffer (minutes) field added to service create/edit form
- Size Category and Coat Type dropdowns added to PetForm (portal)
- @groombook/types Service interface extended with defaultBufferMinutes
- BufferRulesSection embedded in Settings page

The PetForm already had coatType — this commit adds petSizeCategory
and renders both fields with proper dropdown selectors.

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

docs(GRO-1173): add UAT test cases for buffer rules, service buffer, and pet size/coat ... 1820f82cfb

Sections 5.20, 5.21, and 5.22 added to UAT_PLAYBOOK.md:
- TC-WEB-5.20.x: Buffer rules CRUD, validation, empty state (14 cases)
- TC-WEB-5.21.x: Service default buffer field in table and form (7 cases)
- TC-WEB-5.22.x: Pet size category and coat type in portal PetForm (7 cases)

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

Fix: render error state in BufferRulesSection (fixes lint error) ... 5fec0c938a

The 'error' useState was declared but never read — only setError was called.
Now renders the error message as a red text node when the fetch fails.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(CI): use Gitea registry for Docker push ... ef6d9d5ab5

Switch from ghcr.io (GitHub Container Registry) to git.farh.net
Gitea Container Registry. The Gitea Actions runner does not have
access to GitHub's GITHUB_TOKEN for ghcr.io authentication.

Based on fix/ci-registry-auth branch pattern (a582bd0).

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

feat(GRO-1173): add sizeCategory and coatType dropdowns to admin pet form ... 889e1e26ae

- PetForm interface: add sizeCategory and coatType fields
- EMPTY_PET: initialise new fields as empty strings
- openEditPet: pre-populate from pet.petSizeCategory and pet.coatType
- submitPet body: include petSizeCategory and coatType in POST/PATCH
- Pet form UI: add Size Category and Coat Type dropdowns after Breed field
  - Size: Small / Medium / Large / X-Large (maps to enum values)
  - Coat: Smooth / Double / Curly / Wire / Long / Hairless (maps to CoatType union)
  - Both optional — blank "Not set" option matches API optional semantics

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

docs(GRO-1173): add UAT test cases for buffer rules, service default buffer, pet size/coat ... 837b5f6d8a

- §5.7: add TC-WEB-5.7.5 through TC-WEB-5.7.7 for pet size/coat in admin UI
- §5.8.1 (new): add Buffer Rules Management test cases
  - TC-WEB-5.8.2 through TC-WEB-5.8.7: create/edit/delete buffer rules, size/coat filtering, empty state
- §5.9: add TC-WEB-5.9.4 and TC-WEB-5.9.5 for service default buffer minutes

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

Persist pet profile changes via PATCH /api/portal/pets/{petId} ... f958dbdb4f

- handlePetSave is now async; calls PATCH before updating local state
- API response used as source of truth for local state update
- Error state shown on API failure; edit form NOT cleared on failure
- Loading/saving indicator in PetForm while API call in flight

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

docs(GRO-1470): add UAT test cases for pet profile API persistence ... 034f4ab295

Add §5.23 covering:
- API persistence (page reload verification)
- Save error state (form stays open on failure)
- Saving indicator (spinner while in-flight)

Updated UAT_PLAYBOOK.md §5.23

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

Merge pull request 'feat(GRO-1173): admin UI buffer rules, service default buffer, pet size/coat' (#13) from flea-flicker/pet-profile-editor into dev ... f62c0b112d

feat(GRO-1173): admin UI buffer rules, service default buffer, pet size/coat (#13)

Merged-By: The Dogfather (CTO)
Co-Authored-By: Paperclip <noreply@paperclip.ing>

The Dogfather merged commit 62cbfe4e43 into uat 2026-05-21 19:46:41 +00:00

The Dogfather referenced this pull request 2026-05-21 20:25:25 +00:00

feat(GRO-1173): admin UI buffer rules, service default buffer, pet size/coat #13

Sign in to join this conversation.

Reviewers

No Reviewers

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

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/web#14