Skip to content

⚙️ FEATURE-#184: Add NotifyDiscord provider and rename Notify.send to hook_status_task#185

Merged
FernandoCelmer merged 8 commits intodevelopfrom
feature/184
Apr 7, 2026
Merged

⚙️ FEATURE-#184: Add NotifyDiscord provider and rename Notify.send to hook_status_task#185
FernandoCelmer merged 8 commits intodevelopfrom
feature/184

Conversation

@FernandoCelmer
Copy link
Copy Markdown
Member

Description

  • dotflow/abc/notify.py — Rename send() to hook_status_task()
  • dotflow/providers/notify_discord.py — New Discord webhook notification provider
  • dotflow/providers/notify_telegram.py — Refactored with show_result, _build_message, error details
  • dotflow/providers/notify_default.py — Renamed method
  • dotflow/providers/__init__.py — Export NotifyDiscord
  • dotflow/core/task.py — Updated caller to hook_status_task
  • Tests for NotifyDefault, NotifyTelegram, NotifyDiscord
  • Docs updated across README, tutorials, integrations, custom-providers, mkdocs nav

Motivation and Context

Closes #184. Adds Discord webhook support following the same pattern as NotifyTelegram. The send() method was renamed to hook_status_task() to better describe when it is called.

Types of changes

  • Bug fix (change that fixes an issue)
  • New feature (change which adds functionality)
  • Documentation

Checklist

  • I have performed a self-review of my own code
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the CHANGELOG
  • I have updated the documentation accordingly

@FernandoCelmer FernandoCelmer added the enhancement New feature or request label Apr 7, 2026
@FernandoCelmer FernandoCelmer self-assigned this Apr 7, 2026
@FernandoCelmer FernandoCelmer merged commit 1964b82 into develop Apr 7, 2026
10 checks passed
@FernandoCelmer FernandoCelmer deleted the feature/184 branch April 7, 2026 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant