Brohammad / Brohammad/VoxForge

Optional webhook for critical dashboard regression alerts

Open
#49 0 comments 0 reactions 0 assignees View on GitHub
enhancement help wanted
Dominant language
Python
Stars
4
Forks
1
Avg merge
5d 13m
Merged PRs (30d)
10

Description

## Summary

`AlertService` only feeds the dashboard. No outbound Slack/webhook. Handoff docs still list SLA alerts as planned; a webhook is the smallest ops-shaped step.

## Context

- `docs/architecture/alerts.md`
- Dashboard alerts UI (`dashboard/static/app.js`)
- `scripts/uptime-ready-check.sh` Slack notify pattern

## Acceptance criteria

- [ ] Optional `ALERT_WEBHOOK_URL` (Slack incoming-webhook JSON is enough)
- [ ] Fires on new **critical** alerts with org id, code, severity
- [ ] Off by default; no production-validation requirement unless URL set
- [ ] Unit test with mocked HTTP client

Do not build a full notification platform.

Contributor guide

Open the contributing guide

Research direction

Read docs/architecture/alerts.md and inspect the dashboard alert flow in dashboard/static/app.js, then compare the Slack notification pattern in scripts/uptime-ready-check.sh. Locate AlertService and the critical-alert creation path; completion means an unset ALERT_WEBHOOK_URL changes nothing, while a set URL sends the required JSON, with mocked-HTTP unit coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend, observability
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.