groombook/web
13
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

chore: promote dev → uat (GRO-1829 SW fix) #32

Merged
The Dogfather merged 4 commits from dev into uat 2026-05-27 02:27:32 +00:00
Conversation 0 Commits 4 Files Changed 2
+5 -2

4 Commits

Author SHA1 Message Date
Scrubs McBarkley 228a3d746c Merge pull request 'fix(GRO-1829): add skipWaiting/clientsClaim to VitePWA workbox config' (#31) from gro-1829-swpwa-fix into dev
CI / Lint & Typecheck (push) Successful in 17s
Details
CI / Test (push) Successful in 17s
Details
CI / Build & Push Docker Image (push) Successful in 31s
Details
CI / Test (pull_request) Successful in 14s
Details
CI / Lint & Typecheck (pull_request) Successful in 17s
Details
CI / Build & Push Docker Image (pull_request) Successful in 15s
Details
2026-05-27 02:22:25 +00:00
Flea Flicker ad9a178c89 fix: add skipWaiting/clientsClaim to VitePWA workbox config
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 33s
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:22:01 +00:00
The Dogfather 9a3b5d88c8 Merge pull request 'fix(GRO-1822): add role check before /admin redirect — customers access portal' (#30) from fix/gro-1822-role-based-redirect into dev
CI / Test (push) Successful in 20s
Details
CI / Lint & Typecheck (push) Successful in 28s
Details
CI / Build & Push Docker Image (push) Successful in 34s
Details
2026-05-27 01:01:59 +00:00
Flea Flicker 4e487db6f1 fix(GRO-1822): add role check before /admin redirect — customers access portal
CI / Test (pull_request) Failing after 14s
Details
CI / Lint & Typecheck (pull_request) Failing after 17s
Details
CI / Build & Push Docker Image (pull_request) Has been skipped
Details 
App.tsx lines 389-393 redirected ALL authenticated users to /admin,
breaking customer portal access after SSO login.

Now checks `session.user.role === "staff"` before redirecting.
Customers (role !== "staff") can access the portal at /.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-05-27 01:01:28 +00:00