Skip to content

fix(web): clear cached data when someone signs out - #213

Merged
TheGreatAxios merged 4 commits into
mainfrom
cl-12204-clear-cached-data-when-someone-signs-out
Nov 6, 2026
Merged

TheGreatAxios merged 4 commits into
mainfrom
cl-12204-clear-cached-data-when-someone-signs-out

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Signing out now clears everything the interface had saved from the hub, both in memory and in the browser's storage. The next person who signs in on the same browser starts empty and sees only their own projects, instead of the previous person's for a few seconds.

The project list is also only read while someone is signed in, so it stops asking the hub every 30 seconds while nobody is.

Closes CL-12204.

@linear-code

linear-code Bot commented Nov 6, 2026

Copy link
Copy Markdown

CL-12204

@TheGreatAxios
TheGreatAxios marked this pull request as ready for review November 6, 2026 20:17
@corbits-triage

corbits-triage Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Before and after, second account signing in on the same tab right after the first signed out, with hub replies delayed 3 seconds:

Before After
Second account, ~0.3 s after sign-in before after
Signed-out screen before after

Before, the second account's list shows the first account's project; after, it waits for its own read.

Comment thread apps/web/src/queries/client.test.ts Resolved
Comment thread apps/web/src/App.tsx Resolved

@corbits-triage corbits-triage Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No defects reported.

corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 6, 2026
Comment thread apps/web/src/App.tsx Resolved
@TheGreatAxios
TheGreatAxios added this pull request to the merge queue Nov 6, 2026
Merged via the queue into main with commit bede704 Nov 6, 2026
9 checks 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.

1 participant