fix(sync): "Connecting…" only while catching up, not for 30 s after load (#274)
CI / Build & Quality Checks (pull_request) Successful in 1m55s
CI / Trigger Desktop Build (pull_request) Skipped
CI / Docker image build & smoke test (pull_request) Skipped
CI / Secret scan (gitleaks) (pull_request) Successful in 13s
CI / Playwright smoke (e2e) (pull_request) Successful in 13m48s
CI / Build & Quality Checks (pull_request) Successful in 1m55s
CI / Trigger Desktop Build (pull_request) Skipped
CI / Docker image build & smoke test (pull_request) Skipped
CI / Secret scan (gitleaks) (pull_request) Successful in 13s
CI / Playwright smoke (e2e) (pull_request) Successful in 13m48s
The strip showed for PREPARED and for the first SYNCING after it (or after RECONNECTING), i.e. until the next /sync returned: ~30 s of long-poll on a quiet account while the app already worked. A SYNCING means a sync just succeeded, so the strip now shows only during CATCHUP; failures still show as Reconnecting / Connection lost. Fixes #274 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
This commit is contained in:
co-authored by
Claude Opus 5.5
parent
e1168f47bb
commit
c6aac574ee
@@ -65,12 +65,12 @@ export function SyncStatus({ mx }: SyncStatusProps) {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (
|
// [Gitea #274] Only while catching up after a drop. Upstream also showed it
|
||||||
(stateData.current === SyncState.Prepared ||
|
// for PREPARED and for the first SYNCING after PREPARED/RECONNECTING, i.e.
|
||||||
stateData.current === SyncState.Syncing ||
|
// until the NEXT /sync returned: on a quiet account that long-polls ~30 s
|
||||||
stateData.current === SyncState.Catchup) &&
|
// while the app already works (a SYNCING means a sync just succeeded).
|
||||||
stateData.previous !== SyncState.Syncing
|
// A real failure still surfaces as Reconnecting / Connection lost.
|
||||||
) {
|
if (stateData.current === SyncState.Catchup) {
|
||||||
return (
|
return (
|
||||||
<Box direction="Column" shrink="No">
|
<Box direction="Column" shrink="No">
|
||||||
<Box
|
<Box
|
||||||
|
|||||||
Reference in New Issue
Block a user