Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

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

Abierto
  • ¿Algo no está bien?
#635 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
78/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
typescript
Área
backend

Línea de trabajo

Empieza por packages/notifications/src/providers/slack.ts y webhook.ts, y luego inspecciona packages/notifications/src/types.ts, client.ts y las pruebas existentes del provider de Slack. Añade el provider de Discord y la integración de client/type siguiendo el mismo patrón, actualiza packages/notifications/TODO.md para que coincida con el alcance implementado y verifica que pasen las pruebas centradas en Discord.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
TypeScript
Estrellas
1.2k
Forks
227
Merge medio
21 h 11 min
PR fusionados (30 d)
238

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.