Skip to content

[Notifier][Discord] Add DiscordBotTransport #60218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: 7.4
Choose a base branch
from

Conversation

norkunas
Copy link
Contributor

@norkunas norkunas commented Apr 15, 2025

Q A
Branch? 7.4
Bug fix? no
New feature? yes
Deprecations? no
Issues N/A
License MIT

Currently Discord transport is limited because it allows to use only server webhook which sends to a single channel.
I propose to add DiscordBotTransport with a discord+bot scheme, so we can send notifications to multiple channels/as direct messages.

Our use case that we have a channel where all notifications must be sent and then more important ones also to people as direct messages.

@norkunas norkunas requested a review from OskarStark as a code owner April 15, 2025 07:46
@carsonbot carsonbot added this to the 7.3 milestone Apr 15, 2025
@carsonbot carsonbot changed the title [Notifier][Discord] Add DiscordBotTransport [Notifier] [Discord] Add DiscordBotTransport Apr 15, 2025
@norkunas norkunas force-pushed the discord-bot-transport branch from 9016866 to b213dbf Compare April 15, 2025 08:38
@norkunas norkunas force-pushed the discord-bot-transport branch from b213dbf to c7f59ba Compare April 15, 2025 09:14
@fabpot fabpot modified the milestones: 7.3, 7.4 May 26, 2025
@norkunas norkunas force-pushed the discord-bot-transport branch from c7f59ba to f1999a2 Compare May 27, 2025 05:56
@norkunas norkunas force-pushed the discord-bot-transport branch from f1999a2 to d502376 Compare May 27, 2025 05:57
@OskarStark OskarStark changed the title [Notifier] [Discord] Add DiscordBotTransport [Notifier][Discord] Add DiscordBotTransport May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants