Skip to content

feat(nav): alerts bell icon + active-alert badge (#582)#914

Open
vivekchand wants to merge 1 commit intomainfrom
auto/issue-582-alerts-bell-badge
Open

feat(nav): alerts bell icon + active-alert badge (#582)#914
vivekchand wants to merge 1 commit intomainfrom
auto/issue-582-alerts-bell-badge

Conversation

@vivekchand
Copy link
Copy Markdown
Owner

Closes #582

Summary

  • Adds a bell icon button to the nav bar (replacing the commented-out budget button); clicking it navigates to the Alerts tab
  • Adds a red badge on the bell that shows the count of active (unacknowledged) alerts, hidden when count is 0
  • Adds the same badge to the Alerts nav tab, matching the existing Approvals tab pattern
  • Extends the existing checkActiveAlerts() poller in app.js (already runs every 30 s) to update both new badge elements — no new polling interval added

Test plan

  • Start dashboard with no active alerts — bell icon visible, no badge shown, Alerts tab has no badge
  • Trigger an alert (e.g. via the budget monitor or a custom rule) — badge appears on bell and Alerts tab with correct count
  • Dismiss all alerts via the banner — badges disappear within 30 s
  • Count > 99: badge shows "99+" instead of the raw number
  • Clicking the bell navigates to the Alerts tab

Generated by Claude Code

Add a bell icon to the nav bar that shows the count of active
(unacknowledged) alerts from /api/alerts/active, and a matching
badge on the Alerts nav tab — same pattern used by Approvals.

The existing checkActiveAlerts() poller already fetches active
alerts every 30 s; this extends it to update the two new badge
elements on each poll.

Closes #582

Co-Authored-By: Claude <noreply@anthropic.com>
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.

Alerts center: consolidate banners into bell + alerts tab

2 participants