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

fix(gro-1749): sync UAT seed data to root src and fix route path #71

Merged
Flea Flicker merged 3 commits from fix/gro-1749-uat-seed-sync into dev 2026-05-25 17:45:57 +00:00
Conversation 1 Commits 3 Files Changed 3
+71 -6
Flea Flicker commented 2026-05-25 17:42:00 +00:00
Member
Copy Link
Copy Source

Summary

  • Sync UAT Customer/Bella/Max seed data from apps/api/src/routes/admin/seed.ts to root src/routes/admin/seed.ts (the path used by Docker build)
  • Fix route path adminSeedRouter.post("/seed") → adminSeedRouter.post("/") in both files, correcting the endpoint from /api/admin/seed/seed to /api/admin/seed

Changes

src/routes/admin/seed.ts (primary — used by Docker build)

  • Add UAT_CLIENT constant (name: "UAT Customer", email: uat-customer@groombook.dev, phone: 555-0100, status: "active")
  • Add UAT_PETS array: Bella (Poodle, curly) and Max (Labrador Retriever, short)
  • Add UAT Client upsert block after Demo Client
  • Add UAT Pets loop after Demo Dog
  • Fix route: adminSeedRouter.post("/seed") → adminSeedRouter.post("/")

apps/api/src/routes/admin/seed.ts (keep in sync)

  • Same UAT data additions
  • Same route path fix

Test plan

  • POST /api/admin/seed returns 200 with UAT Customer and Bella/Max in details array
  • Running twice returns "already exists" messages (idempotent)

cc @cpfarhood

## Summary - Sync UAT Customer/Bella/Max seed data from `apps/api/src/routes/admin/seed.ts` to root `src/routes/admin/seed.ts` (the path used by Docker build) - Fix route path `adminSeedRouter.post("/seed")` → `adminSeedRouter.post("/")` in both files, correcting the endpoint from `/api/admin/seed/seed` to `/api/admin/seed` ## Changes ### `src/routes/admin/seed.ts` (primary — used by Docker build) - Add `UAT_CLIENT` constant (name: "UAT Customer", email: uat-customer@groombook.dev, phone: 555-0100, status: "active") - Add `UAT_PETS` array: Bella (Poodle, curly) and Max (Labrador Retriever, short) - Add UAT Client upsert block after Demo Client - Add UAT Pets loop after Demo Dog - Fix route: `adminSeedRouter.post("/seed")` → `adminSeedRouter.post("/")` ### `apps/api/src/routes/admin/seed.ts` (keep in sync) - Same UAT data additions - Same route path fix ## Test plan - [ ] `POST /api/admin/seed` returns 200 with UAT Customer and Bella/Max in `details` array - [ ] Running twice returns "already exists" messages (idempotent) cc @cpfarhood
Flea Flicker added 2 commits 2026-05-25 17:42:02 +00:00
fix(ci): add provenance: false to fix registry push failures
CI / Test (pull_request) Successful in 1m6s
Details
CI / Lint & Typecheck (pull_request) Successful in 1m10s
Details
CI / Build & Push Docker Images (pull_request) Successful in 37s
Details
d17915907c 
OCI attestation manifests fail to push on Gitea registry when image
layers already exist (cross-repo blob references). Disabling provenance
generation on all four build-push-action steps ensures a simple single
manifest push that Gitea handles reliably.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
fix(gro-1749): sync UAT seed data to root src and fix route path
CI / Lint & Typecheck (pull_request) Successful in 12s
Details
CI / Test (pull_request) Successful in 12s
Details
CI / Build & Push Docker Images (pull_request) Successful in 1m11s
Details
a848a3f35f 
- Add UAT_CLIENT (name: "UAT Customer", email: uat-customer@groombook.dev)
- Add UAT_PETS: Bella (Poodle, curly) and Max (Labrador Retriever, short)
- Add UAT Client upsert block and UAT Pets loop after Demo Dog
- Change adminSeedRouter.post("/seed") → adminSeedRouter.post("/")
  in both src/routes/admin/seed.ts (Docker build) and apps/api/src/

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
The Dogfather added 1 commit 2026-05-25 17:45:37 +00:00
Merge origin/dev into fix/gro-1749-uat-seed-sync 3bc8c634ab 
Resolve conflict in apps/api/src/routes/admin/seed.ts:
- Keep UAT_CLIENT with address (not stripped by GRO-1743)
- Keep weightKg in UAT_PETS (from both branches)
- Use origin/dev coatType as const (typed properly)
- Use existingPet naming (from origin/dev)

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Flea Flicker merged commit a03771f7e7 into dev 2026-05-25 17:45:57 +00:00
Flea Flicker commented 2026-05-25 18:05:51 +00:00
Author
Member
Copy Link
Copy Source

GRO-1750 Progress

  • PR #72 (dev→uat) merged — uat now has commit a03771f (GRO-1749)
  • Infra PR #527 created targeting main with new image tag e5f16a5
  • Infra PR #527 merge pending CI pass

See GRO-1750 for full tracking.

## GRO-1750 Progress - [x] PR #72 (dev→uat) merged — uat now has commit a03771f (GRO-1749) - [x] Infra PR #527 created targeting main with new image tag e5f16a5 - [ ] Infra PR #527 merge pending CI pass See GRO-1750 for full tracking.
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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/api#71
Delete Branch "fix/gro-1749-uat-seed-sync"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?