groombook/app
Archived
13
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(portal): remove sessionAttempted from redirect condition (GRO-309)

Browse Source
This commit is contained in:
Barkley Trimsworth
2026-03-31 18:45:08 +00:00
parent d4bdca5616
commit df32509186
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/web/src/portal/CustomerPortal.tsx
+5 -5
View File
@@ -184,12 +184,12 @@ export function CustomerPortal() {
const avatarInitials = (clientName.split(" ")[0] || "G").charAt(0).toUpperCase(); const avatarInitials = (clientName.split(" ")[0] || "G").charAt(0).toUpperCase();
// After init completes, redirect unauthenticated users to /login and staff to /admin // After init completes, redirect unauthenticated users to /login and staff to /admin.
// The portal chrome must NEVER be visible to users without a valid client session. // The portal chrome must NEVER be visible to users without a valid client session.
// Only redirect if we have NOT attempted a session fetch yet — if a fetch is in-flight // We check !session rather than sessionAttempted because a failed session fetch still
// (E2E mock resolves synchronously, batched with setInitComplete), sessionAttempted // means we must redirect — sessionAttempted being true only means we attempted to
// is still false so we don't redirect prematurely. // create a session, not that one exists.
if (initComplete && !session && !sessionAttempted) { if (initComplete && !session) {
const devUser = getDevUser(); const devUser = getDevUser();
if (devUser && devUser.type === "staff") { if (devUser && devUser.type === "staff") {
return <Navigate to="/admin" replace />; return <Navigate to="/admin" replace />;