Skip to content

Fix/dashboard polling backoff#2054

Open
shravanithouta108 wants to merge 6 commits into
utksh1:mainfrom
shravanithouta108:fix/dashboard-polling-backoff
Open

Fix/dashboard polling backoff#2054
shravanithouta108 wants to merge 6 commits into
utksh1:mainfrom
shravanithouta108:fix/dashboard-polling-backoff

Conversation

@shravanithouta108

Copy link
Copy Markdown
Contributor

Description

Dashboard.tsx was polling the backend every 10 seconds indefinitely, even when the backend was completely down. This caused unnecessary network requests, stale summary data persisting silently, and no way to recover without a full page reload.

Fixed by adding a healthFailed state flag that stops polling immediately after the first health check failure. The Retry Connection button now resets this flag and restarts polling cleanly instead of reloading the entire page.

Related Issues

Closes #1852

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Verified polling stops after backend health check fails
  • Verified error state is shown immediately when backend is unreachable
  • Verified Retry Connection button restarts polling without full page reload
  • Verified normal polling continues as expected when backend is available
  • Ran frontend checks: npm run test — all passed

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

- Replace fixed sm:grid-cols-2 lg:grid-cols-3 xl:grid-cols-4 with auto-fit minmax(180px,1fr)
- Ensures each filter item has minimum 180px width
- Controls wrap gracefully to new line when horizontal space is insufficient
- Fixes overlapping TARGET, SCANNER, ASSET, FINDING KIND, ANALYST STATE filters
@utksh1 utksh1 added level:intermediate 35 pts difficulty label for moderate contributor PRs type:bug Bug fix work category bonus label area:frontend Frontend React/UI work labels Jul 24, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This dashboard polling change is stacked on the unrelated AI-summary, reporting, and Findings edits from #2043. Please rebase onto current main with only Dashboard.tsx plus focused polling/backoff/retry coverage, then rerun checks.

@utksh1 utksh1 added type:feature Feature work category bonus label area:backend Backend API, database, or service work labels Jul 24, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This branch combines the already-merged Findings layout change with AI summary behavior and Dashboard polling. Please split it into a focused Dashboard polling PR, remove unrelated files, and rebase before re-review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work area:frontend Frontend React/UI work level:intermediate 35 pts difficulty label for moderate contributor PRs type:bug Bug fix work category bonus label type:feature Feature work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[#78] Dashboard 10s polling never backs off on error

2 participants