Skip to content

workplace: the room is read once at boot whether or not the tab is visible #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
richard-epsilla merged 1 commit into from
Oct 2, 2026
Merged

workplace: the room is read once at boot whether or not the tab is visible #40

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions kits/workplace/app/src/App.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,12 @@ function Workplace() {
}, [me, harnessIds]);

const ready = !!rec && !!me;
// The first read of the room is unconditional: the periodic refreshes below run only while the
// tab is visible, and a tab that booted in the background (opened behind another, or switched
// away from in its first seconds) sat on "Opening the workplace" with an empty room list until
// it was looked at (2026-10-02).
useEffect(() => { if (ready) refreshTeammates(); }, [ready, refreshTeammates]);
useEffect(() => { if (ready && teammates) refreshCards(); }, [ready, teammates !== null, refreshCards]); // eslint-disable-line react-hooks/exhaustive-deps
useVisibleInterval(() => { if (ready) refreshTeammates(); }, TEAMMATES_MS, [ready, refreshTeammates]);
useVisibleInterval(() => { if (ready && teammates) refreshCards(); }, CARDS_MS, [ready, teammates !== null, refreshCards]);

Expand Down
Loading