groombook/web

fix(App.tsx): check user role before redirecting to /admin #29

Closed

The Dogfather wants to merge 3 commits from ccfa5281-2076-40c2-87a9-bf2dbcf98d22/gro-1822-role-based-redirect into dev

Author	SHA1	Message	Date
Flea Flicker	85294b108d	fix: add skipWaiting/clientsClaim to VitePWA workbox config CI / Test (pull_request) Successful in 13s Details CI / Lint & Typecheck (pull_request) Successful in 25s Details CI / Build & Push Docker Image (pull_request) Successful in 50s Details Root cause: SW remained in waiting phase after redeploy, serving stale precached assets. Without skipWaiting/clientsClaim the old SW persisted and controlled the page even after a new SW was installed. Fixes blank-page regression where React never mounted on login.	2026-05-27 02:20:41 +00:00
Flea Flicker	4213c1f2e7	docs(UAT_PLAYBOOK.md): add TC-WEB-SSO-ROLE-* test cases for GRO-1822 CI / Test (pull_request) Successful in 15s Details CI / Lint & Typecheck (pull_request) Successful in 17s Details CI / Build & Push Docker Image (pull_request) Successful in 34s Details Add section 5.4.3 covering role-based redirect after SSO login: - Customer SSO → portal at / (not redirected to /admin) - Staff SSO → redirect to /admin - Impersonation bypass via ?sessionId= preserved - Dev mode unaffected Refs: GRO-1822	2026-05-27 00:54:07 +00:00
Flea Flicker	505904d8bd	fix(App.tsx): check user role before redirecting to /admin CI / Test (pull_request) Successful in 22s Details CI / Lint & Typecheck (pull_request) Successful in 26s Details CI / Build & Push Docker Image (pull_request) Successful in 45s Details - Staff users (role !== "customer") continue to redirect to /admin - Customer users (role === "customer") see the portal at / instead - Impersonation flow via ?sessionId= remains unaffected - Dev mode (authDisabled=true) unchanged Refs: GRO-1822	2026-05-27 00:53:16 +00:00