Skip to content

fix: refresh profile after email verification - #27

Open
bluewandd wants to merge 1 commit into
aervxa:mainfrom
bluewandd:pr/lepse-email-verification
Open

fix: refresh profile after email verification#27
bluewandd wants to merge 1 commit into
aervxa:mainfrom
bluewandd:pr/lepse-email-verification

Conversation

@bluewandd

@bluewandd bluewandd commented Sep 2, 2026

Copy link
Copy Markdown

An already-open Lachesis window could keep showing the email-verification banner after verification completed elsewhere. The server updated the profile, but the cached response was not refreshed on window activation.

The profile query now refreshes on visibility or window focus only while its cached profile is not verified. Once verification succeeds, later focus returns generate no profile requests. Startup uses the canonical profile key and skips profile requests when there is no session, avoiding the previous anonymous request and refreshing persisted profile data with one request.

The token checks are request guards only; they are not cache isolation and this change does not claim to clear or hide hydrated data. Ordinary query defaults remain unchanged.

A focused Query harness confirmed the normal false/true states, malformed cached data, request counts, and absence of unhandled rejections. Formatting and diff checks pass.

@bluewandd

bluewandd commented Sep 2, 2026

Copy link
Copy Markdown
Author

Superseded. This PR has been rewritten as a single independent change against current main, and no longer belongs to a sequence or requires any particular merge order.

The earlier branch was based on 7e8c988 and is gone. The new commit is 3 files, +25/-5, and touches only the profile query lifecycle, the focus listener, and the startup prefetch. See the updated description.

@bluewandd
bluewandd force-pushed the pr/lepse-email-verification branch from 260fe99 to 5a576a7 Compare September 6, 2026 07:34
@bluewandd bluewandd changed the title feat: add email verification fix: refresh profile after email verification Sep 6, 2026
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.

2 participants