Skip to content

Notifications to Discord #1069

@yanivbh1

Description

@yanivbh1
Contributor

Description

Add support in pushing Memphis notifications to Discord with configuration through the Integration Center.

Important to know

  • To avoid rate limitations, a buffer should be implemented and push notifications in batches.

Activity

Avitaltrifsik

Avitaltrifsik commented on Jul 16, 2023

@Avitaltrifsik
Contributor

Hey @yanivbh1 could you please add a description to this issue?

yanivbh1

yanivbh1 commented on Jul 16, 2023

@yanivbh1
ContributorAuthor

@Avitaltrifsik should be ok.

StarscourgeVelkhana

StarscourgeVelkhana commented on Jul 17, 2023

@StarscourgeVelkhana

@yanivbh1
Hey Yaniv. I'm currently on this issue and would need some more information where the discord notifications could be sent. Options I think of:

  • Send discord notification as a private message
  • Send discord notification to a server channel
  • Send discord notifications to multiple server channels
  • Send discord notification to multiple servers and specified channels

One more question:
Where do you guys store sensitive data. I will have to integrate the bot token somewhere.

yanivbh1

yanivbh1 commented on Jul 17, 2023

@yanivbh1
ContributorAuthor

Hey @StarscourgeVelkhana ,
I would say either private or server channels. Multiple would be preferred.
Creds are saved encrypted within the metadata store, which is a vanilla Postgres.

StarscourgeVelkhana

StarscourgeVelkhana commented on Jul 17, 2023

@StarscourgeVelkhana

Alright.
Only multiple channels or also multiple servers (and channels)?

What I mean: Can the bot be added only to one server or multiple

yanivbh1

yanivbh1 commented on Jul 17, 2023

@yanivbh1
ContributorAuthor

Single server, multiple channels.
Thank you for your help @StarscourgeVelkhana !

turulix

turulix commented on Jul 31, 2023

@turulix
Contributor

Wouldn't it be easier to just use Webhooks instead of making an entire bot for it?

yanivbh1

yanivbh1 commented on Jul 31, 2023

@yanivbh1
ContributorAuthor

That’s a much better idea @turulix

yanivbh1

yanivbh1 commented on Jul 31, 2023

@yanivbh1
ContributorAuthor

@StarscourgeVelkhana what do you think?

yanivbh1

yanivbh1 commented on Aug 12, 2023

@yanivbh1
ContributorAuthor

The feature is available for contribution.

linked a pull request that will close this issue on Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @turulix@yanivbh1@Avitaltrifsik@StarscourgeVelkhana

      Issue actions

        Notifications to Discord · Issue #1069 · superstreamlabs/memphis