Lint Roller gb_lint
  • Joined on 2026-05-17
Lint Roller approved groombook/web#78 2026-06-11 18:31:11 +00:00
Promote dev → uat: GRO-2373 in-portal chrome sign-out button

QA approved: all acceptance criteria verified.

Lint Roller approved groombook/api#213 2026-06-11 16:42:05 +00:00
Promote dev → uat: GRO-2359 clients-from-auth endpoint

QA approved. CI green (lint, typecheck, test, build). Code review PASS.

Lint Roller approved groombook/web#76 2026-06-11 16:42:00 +00:00
Promote dev → uat: GRO-2359 OOBE portal-creation routing

QA approved. CI green (lint, typecheck, test, build). Code review PASS.

Lint Roller approved groombook/web#73 2026-06-11 14:31:30 +00:00
Promote dev → uat: GRO-2358 logout on no-access screen

LGTM. Code, tests, and UAT_PLAYBOOK all correct. CI green. Approving for self-merge.

Lint Roller approved groombook/api#209 2026-06-10 09:21:51 +00:00
Promote dev → uat: GRO-2342 portal waitlist service {id, name}

QA approved. CI clean (lint+typecheck, test, build all pass). Code review: GRO-2342 service lookup batched correctly, shape consistent across appointment and waitlist cards, guard w.serviceId correct. Tests cover both card shapes cleanly. GRO-2139 advisory lock wraps full drop->migrate->seed chain, pool max:6 load-bearing. UAT_PLAYBOOK.md TC-API-8.20 present. GRO-2294 carry-forward plumbing in scope. LGTM — self-merge.

Lint Roller approved groombook/api#205 2026-06-09 10:46:59 +00:00
feat(GRO-2319): dev→uat — portal waitlist surfacing + seed (api)

QA approved (GRO-2331). CI green (Lint, Test, Docker all success). Diff scoped to GRO-2319 only. UAT_PLAYBOOK TC-API-8.19 added. GET /portal/appointments surfaces active waitlist rows as synthetic waitlisted cards correctly. Seed upsert is idempotent. Tests cover presence and absence cases.

Lint Roller approved groombook/web#70 2026-06-09 10:46:58 +00:00
feat(GRO-2319): dev→uat — live StatusBadge palette (web)

QA approved (GRO-2331). CI green (Lint, Test, Docker all success). Diff scoped to GRO-2319 only. UAT_PLAYBOOK §5.12f (TC-WEB-5.12.26–29) updated. normalizeStatusKey no_show→no-show, deriveDisplayStatus, and waitlist card styling/actions logic correct. Tests cover all new paths.

Lint Roller approved groombook/api#200 2026-06-09 09:11:31 +00:00
dev → uat: GRO-2172 extended pet fields

LGTM. Code review PASS: all 4 fields match spec, medicalAlerts jsonb cast correct, updatePetSchema via .partial() correct, UAT_PLAYBOOK.md §3 TC-API-3.8→3.15 already documented. CI: Test ✓, Lint & Typecheck ✓ (first L&TC was a 12m runner flake; rerun passed).

Lint Roller approved groombook/api#196 2026-06-09 06:57:07 +00:00
dev → uat: GRO-2299 redact googleMapsApiKey from PATCH /api/admin/settings

Code review passed. All CI green (Test, Lint/Typecheck, Docker).

Lint Roller approved groombook/api#194 2026-06-09 06:24:41 +00:00
Promote GRO-2294 to UAT: Route Optimization security hardening

LGTM. Both changes are clean and correct:

Lint Roller approved groombook/web#67 2026-06-09 04:38:06 +00:00
Promote dev → uat: GRO-2160 route nav export + offline polish

QA approved. All 10 checklist items verified: export buttons conditional on stops, iOS/Android prominence correct, error path closes pre-opened tab + shows inline error, offline caching correct (NetworkFirst api-cache + CacheFirst osm-tiles), tile pre-warm on load/optimize/reorder, responsive layout single-column ≤768px, UAT_PLAYBOOK.md §5.30 present and matches implementation, CI green.

Lint Roller approved groombook/web#64 2026-06-09 03:04:22 +00:00
Promote dev → uat: GRO-2159 drag-to-reorder + re-optimize

QA review complete — LGTM.

Lint Roller approved groombook/web#61 2026-06-09 01:56:05 +00:00
Promote dev → uat: GRO-2158 route planner page

QA Review — APPROVED

Lint Roller merged pull request groombook/api#188 2026-06-09 00:26:19 +00:00
Promote dev→uat: GRO-2225 + GRO-2235 + GRO-2157 (atomic)
Lint Roller pushed to uat at groombook/api 2026-06-09 00:26:19 +00:00
4868f18dfd Merge pull request 'Promote dev→uat: GRO-2225 + GRO-2235 + GRO-2157 (atomic)' (#188) from promote/dev-to-uat-gro-2225 into uat
37e42b3104 ci: re-trigger checks (transient pnpm/action-setup runner flake)
d617c69571 Merge remote-tracking branch 'origin/dev' into promote/dev-to-uat-gro-2225
cd2f60e282 feat(GRO-2157): navigation export endpoints (Phase 2.3) (#190)
6702086c7b fix(GRO-2235): return 409 on duplicate portal waitlist submit (#189)
Compare 7 commits »
Lint Roller approved groombook/api#188 2026-06-09 00:26:12 +00:00
Promote dev→uat: GRO-2225 + GRO-2235 + GRO-2157 (atomic)

Code review passed — GRO-2157/2235/2225 all clean.

Lint Roller approved groombook/web#56 2026-06-08 19:25:00 +00:00
Promote dev → uat: GRO-2211/2218/2207 + GRO-2234 portal Book New (cumulative)

QA approved — GRO-2241 review pass. GRO-2234 handleConfirmBooking: 401 → remint → one retry with fresh session id. Header sequence test confirmed. CI green. UAT_PLAYBOOK §5.12e TC-WEB-5.12.25 confirmed. Self-merge cleared.

Lint Roller approved groombook/api#182 2026-06-08 19:24:59 +00:00
Promote dev → uat: GRO-2155/2156/2203/2211/2163 + GRO-2234 (cumulative batch)

QA approved — GRO-2241 review pass. GRO-2234 validatePortalSession: SSO-bridge scoped sliding TTL, 30-min idle/8h cap, expired sessions blocked before slide, write-throttle present. 6 test cases clean. CI green. UAT_PLAYBOOK §8.17 confirmed. Self-merge cleared.

Lint Roller approved groombook/api#182 2026-06-08 18:18:51 +00:00
Promote dev → uat: GRO-2155/2156/2203/2211/2163 + GRO-2234 (cumulative batch)

LGTM — code review complete (GRO-2237). Travel buffer + conflict detection: correct first-stop-zero rule, null-travel-as-zero, overlapping appointment handling, pure flag-only (no auto-move). recomputeLegsForOrder correctly uses offline heuristic for manual reorder. Two-pass stopOrder update avoids unique-constraint collision. Permutation validation exhaustive. Buffer re-applied correctly across all three endpoints. 7 unit tests cover all edge cases. CI green (Test / Lint & Typecheck / Build). UAT playbook §4.16-4.17 complete.