Skip to content

fix(web): remove logout route#1794

Merged
tyler-dane merged 2 commits into
SwitchbackTech:mainfrom
GHX5T-SOL:fix/remove-logout-route-1793
May 24, 2026
Merged

fix(web): remove logout route#1794
tyler-dane merged 2 commits into
SwitchbackTech:mainfrom
GHX5T-SOL:fix/remove-logout-route-1793

Conversation

@GHX5T-SOL
Copy link
Copy Markdown
Contributor

@GHX5T-SOL GHX5T-SOL commented May 19, 2026

Summary

  • remove the dedicated /logout route and view
  • make authenticated Z shortcut logout immediately
  • make command-palette logout actions call the same logout path directly
  • isolate logout tests so they exercise the real logout hook instead of competing alias mocks

Closes #1793

Validation

  • bun test --cwd packages/web src/auth/compass/hooks/useLogout.test.ts src/common/hooks/useLogoutCmdItems.test.ts src/views/Week/hooks/shortcuts/useGlobalShortcuts.test.tsx (7 pass)
  • bun test:web (passed)
  • bun test:core (passed)
  • bun test:backend (passed when rerun alone; the first parallel run with scripts collided on Jest MongoDB globalConfig cleanup)
  • bun test:scripts (passed)
  • bun type-check (passed)
  • bun lint (passed)
  • git diff --check (passed)

Notes

  • GitHub currently reports no checks for this branch via gh pr checks 1794.

@tyler-dane tyler-dane force-pushed the fix/remove-logout-route-1793 branch from c47521d to 8268ae2 Compare May 24, 2026 00:18
@tyler-dane tyler-dane force-pushed the fix/remove-logout-route-1793 branch from 8268ae2 to 10c463d Compare May 24, 2026 00:35
@tyler-dane tyler-dane merged commit 3f68ac0 into SwitchbackTech:main May 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove /logout route

2 participants