Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

docs(notifications): TODO.md claims Discord/Teams/Telegram/Google Chat providers exist — they don't

Aperta
  • Qualcosa non va?
#635 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
78/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
typescript
Ambito
backend

Direzione di ricerca

Inizia da packages/notifications/src/providers/slack.ts e webhook.ts, poi esamina packages/notifications/src/types.ts, client.ts e i test esistenti del provider Slack. Aggiungi il provider Discord e il wiring di client e tipi seguendo lo stesso modello, aggiorna packages/notifications/TODO.md in modo che corrisponda all'ambito implementato e verifica che i test incentrati su Discord passino.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Summary

packages/notifications/TODO.md checks off Discord, Microsoft Teams, Telegram, and Google Chat as [x] done:

- [x] **Slack** - Webhook support
- [x] **Discord** - Webhook support
- [x] **Email** - Injected send function
- [x] **Webhook** - Generic HTTP webhook
- [x] **Microsoft Teams** - Adaptive Cards via webhook
- [x] **Telegram** - Bot API (sendMessage)
- [x] **Google Chat** - Cards via webhook

But packages/notifications/src/providers/ only contains slack.ts, email.ts, webhook.ts, and base.ts — no matching files for the other four. NotificationChannel (packages/notifications/src/types.ts:1) is typed "slack" | "email" | "webhook" only, and NotificationClient (packages/notifications/src/client.ts) only wires those three up. The alarms-system section of the same TODO.md (line 13) still lists discord/teams/telegram/google-chat as channels an alarm should be able to target, so this looks like real missing functionality rather than just stale docs.

Proposal

Add a Discord provider (webhook-based, mirroring the existing SlackProvider) as a first, self-contained slice:

  • New providers/discord.ts following the SlackProvider/WebhookProvider pattern
  • Extend NotificationChannel and wire it into NotificationClient
  • Tests mirroring the existing Slack provider tests
  • Fix the TODO.md checkboxes to reflect reality

Keeping this scoped to Discord only — happy to do Teams/Telegram/Google Chat as separate follow-up PRs if this shape is right, rather than one big PR.

Checking scope/interest before opening a PR, per the AI policy here — let me know if this is wanted or if there's context I'm missing (e.g. planned elsewhere / superseded by a different design).

Disclosure: I used Claude Code to grep the providers directory against TODO.md and the type definitions; verified manually before writing this up.

Lingua principale
TypeScript
Stelle
1.2k
Fork
227
Merge medio
21h 11m
PR unite (30g)
238

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di databuddy-analytics/Databuddy

Tutte le issue di databuddy-analytics/Databuddy

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.