Skip to content

fix(dashboard): refresh after manual sync failure - #149

Merged
pitimon merged 1 commit into
mainfrom
fix/146-manual-refresh-survives-sync-failure
Aug 5, 2026
Merged

fix(dashboard): refresh after manual sync failure#149
pitimon merged 1 commit into
mainfrom
fix/146-manual-refresh-survives-sync-failure

Conversation

@pitimon

@pitimon pitimon commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • isolate local sync failure handling inside manual refresh
  • continue refreshing every dashboard panel after sync failure
  • keep manual usage-limits refresh forced while scheduled refresh remains cache-aware
  • log sync failure separately from downstream refresh failure

Verification

  • strict RED: sync rejection cancelled all panel refreshes before the fix
  • GREEN: all refresh hooks run once after sync rejection, including forced usage limits
  • successful sync and scheduled interval/post-sync semantics remain covered
  • focused dashboard tests: 15/15 pass
  • independent review: PASS, no findings
  • post-rebase npm run ci:local: dashboard 321/321, root 1037/1037, all validators green
  • changed-file ESLint and static scan pass

Closes #146

@pitimon
pitimon force-pushed the fix/146-manual-refresh-survives-sync-failure branch from a966634 to 0e67bd9 Compare August 5, 2026 09:05
@pitimon
pitimon merged commit 7e01911 into main Aug 5, 2026
1 check passed
@pitimon
pitimon deleted the fix/146-manual-refresh-survives-sync-failure branch August 5, 2026 09:07
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.

dashboard: manual refresh is silently cancelled when local sync fails

1 participant