groombook/api
13
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(GRO-1470): add portal PATCH /pets/:id + expand GET /pets response #40

Merged
The Dogfather merged 3 commits from flea-flicker/gro-1162-pet-buffer-time into dev 2026-05-21 20:16:53 +00:00
Conversation 1 Commits 3 Files Changed 7
+201 -5

3 Commits

Author SHA1 Message Date
Flea Flicker 44da26820b feat(GRO-1171): add Admin API — Buffer Rules CRUD + service/pet updates
CI / Lint & Typecheck (pull_request) Failing after 7s
Details
CI / Test (pull_request) Failing after 9s
Details
CI / Build & Push Docker Image (pull_request) Has been skipped
Details 
- Add buffer_rules table with serviceId/sizeCategory/coatType/bufferMinutes
- Add petSizeCategoryEnum (small/medium/large/extra_large) and coatTypeEnum
  to schema; update pets table columns to use the typed enums
- Add defaultBufferMinutes to services table
- Add apps/api/src/routes/buffer-rules.ts with GET/POST/PATCH/DELETE,
  all manager-only via requireRole("manager")
- Register /api/buffer-rules router in index.ts
- PATCH /api/services/:id accepts optional defaultBufferMinutes
- POST/PATCH /api/pets accepts optional sizeCategory and coatType

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-05-21 20:10:20 +00:00
Flea Flicker 21981fbdc4 fix(GRO-1365): add missing imports for and/eq/exists/or in test 
The vi.mock factory uses db.and/eq/exists/or from the imported module,
but TypeScript's module-level import binding (const declarations)
can't be referenced inside the async factory before initialization.
Adding top-level imports from "../db/index.js" and using them
directly in the mock return fixes the TDZ error.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-05-21 20:10:20 +00:00
Chris Farhood 85fc803548 fix(GRO-1365): address QA review findings on api/#21 
1. Fix vi.mock factory: importOriginal -> db.and/eq/exists/or stubs
   (removes ReferenceError from undeclared imports in test)
2. Remove MedicalAlert.id — not in schema/migration/DB, only in types
3. Replace z.string().max(100) coatType with z.enum for CoatType union
4. Fix test expecting coatType "smooth" (invalid) -> "double" (valid)
5. Add TC-API-3.8 through TC-API-3.15 to UAT_PLAYBOOK.md §4.3

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-21 20:10:20 +00:00