skatespots/SkateSpots
1
0
Fork 0
Code Issues Pull requests Projects Releases 9 Packages Wiki Activity Actions 1

fix(cordn): keep the invite poll alive across re-renders

Browse source
This commit is contained in:
SkateSpots 2026-10-27 16:43:49 +07:00
parent 7a7b1c4950
commit 5ac9e67860
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/components/crew/CordnNativePanel.tsx
View file

@ -214,6 +214,13 @@ export function CordnNativePanel({ crew, canManage, isMember }: CordnNativePanel
if (!awaiting) setNotice(t('cordnUi.noInvite')); if (!awaiting) setNotice(t('cordnUi.noInvite'));
}, [checkNow, awaiting, t, native.busy]); }, [checkNow, awaiting, t, native.busy]);
// Keep the latest checker in a ref: the poll must not restart just because a
// parent re-render gave the hook callbacks new identities.
const checkNowRef = useRef(checkNow);
useEffect(() => {
checkNowRef.current = checkNow;
}, [checkNow]);
// Poll the coordinator while a join request is pending. Backs off from 10s to // Poll the coordinator while a join request is pending. Backs off from 10s to
// 30s so a founder who approves later is picked up without a manual tap. // 30s so a founder who approves later is picked up without a manual tap.
// Interactive signers (Amber) are excluded — polling would pop their signer. // Interactive signers (Amber) are excluded — polling would pop their signer.
@ -225,7 +232,7 @@ export function CordnNativePanel({ crew, canManage, isMember }: CordnNativePanel
const tick = async () => { const tick = async () => {
if (cancelled) return; if (cancelled) return;
const joined = await checkNow(); const joined = await checkNowRef.current();
if (cancelled) return; if (cancelled) return;
if (joined) { if (joined) {
setAwaiting(false); setAwaiting(false);
@ -241,7 +248,7 @@ export function CordnNativePanel({ crew, canManage, isMember }: CordnNativePanel
cancelled = true; cancelled = true;
if (timer !== undefined) window.clearTimeout(timer); if (timer !== undefined) window.clearTimeout(timer);
}; };
}, [awaiting, native.group, native.ready, native.interactiveHintVisible, checkNow]); }, [awaiting, native.group, native.ready, native.interactiveHintVisible]);
// "Join our crew chat" invite. The chat needs the person to be a crew member // "Join our crew chat" invite. The chat needs the person to be a crew member
// first, so the link just drops them on this crew's private-chat tab with the // first, so the link just drops them on this crew's private-chat tab with the