Skip to content

Alert UX: preview text, SMTP validation, CPU default note (#109)#125

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/alert-ux-round2
Feb 19, 2026
Merged

Alert UX: preview text, SMTP validation, CPU default note (#109)#125
erikdarlingdata merged 1 commit intodevfrom
feature/alert-ux-round2

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Live preview text below alert thresholds shows exactly what's enabled: "Will alert when: blocking > 30s, CPU > 90%, ..."
  • "Validate Settings" button on SMTP config checks for empty/invalid fields before you need to send a test email
  • Lite CPU threshold row now explains why the default is 80% (Lite collects directly from DMVs, catches spikes faster)
  • Preview text updates dynamically as checkboxes and thresholds change

Test plan

  • Dashboard Settings > Notifications: verify preview text updates as you toggle checkboxes and change values
  • Dashboard Settings > Email Alerts: click "Validate Settings" with empty fields, verify warning
  • Dashboard Settings > Email Alerts: fill all fields, click "Validate Settings", verify success message
  • Lite Settings > Notifications: verify preview text and CPU explanation note
  • Lite Settings > Email Alerts: verify "Validate Settings" button works

🤖 Generated with Claude Code

…109)

- Add live preview text below alert thresholds: "Will alert when: blocking > 30s, CPU > 90%, ..."
- Add "Validate Settings" button to SMTP config in both Dashboard and Lite
- Add explanation for Lite's 80% CPU default vs Dashboard's 90%
- Preview text updates dynamically as checkboxes/thresholds change

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata mentioned this pull request Feb 18, 2026
12 tasks
@erikdarlingdata erikdarlingdata merged commit 186d449 into dev Feb 19, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the feature/alert-ux-round2 branch February 20, 2026 13:20
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.

1 participant