Skip to content
This repository has been archived by the owner on May 29, 2024. It is now read-only.

Telegram bot notification support #184

Open
ghost opened this issue Nov 5, 2023 · 9 comments · May be fixed by #206
Open

Telegram bot notification support #184

ghost opened this issue Nov 5, 2023 · 9 comments · May be fixed by #206
Assignees
Labels
type: enhancement New feature or request

Comments

@ghost
Copy link

ghost commented Nov 5, 2023

It would be also great if we could alert certain events via Telegram asides Slack as well

@epociask
Copy link
Collaborator

epociask commented Nov 7, 2023

Hey thank you so much for filing an issue. This would be cool to have but isn't particularly a top priority for us at the moment. You're welcome to do the implementation yourself if you'd like, I can happily provide you resources to get started! Otherwise we'd need to see more community interest before committing to an implementation on our end :)

@epociask epociask added the type: enhancement New feature or request label Nov 8, 2023
@richardgreg
Copy link
Contributor

Hi @epociask, would love to take this up 🙂

@epociask
Copy link
Collaborator

Hi @richardgreg, sorry for just getting back. I've assigned it to you! Please lmk if you have any questions 🙏🏻

@richardgreg
Copy link
Contributor

No problem. You mentioned providing resources to get started. I'd love any resource you have to provide 🙂

@epociask
Copy link
Collaborator

epociask commented Nov 30, 2023

@richardgreg
Absolutely! You'll need to likely do the following to enable this:

  1. Adding a client construct in the client package
  2. Adding enums and support for client configurations in a alert-routing.yaml definition (see alert-routing)
  3. Supporting a test server implementation in the e2e framework

Lmk if you have any questions at all. My tg is @epcoiask if you'd like to ping me!

@adelowo
Copy link
Contributor

adelowo commented Feb 13, 2024

@epociask do you have an idea if someone internally is working on this? I just might need this over the next week or so
and will be open to contributing a fix

@epociask
Copy link
Collaborator

Hey @adelowo, I don't believe anyone is working on this atm. Feel free to go forward with an implementation if you'd like.
Otherwise I can spend some time on it over the next week or two 🙏🏻

@richardgreg
Copy link
Contributor

@epociask do you have an idea if someone internally is working on this? I just might need this over the next week or so and will be open to contributing a fix

Working on this too. I'm on track to submit a working solution before the end of the month

@richardgreg richardgreg linked a pull request Feb 14, 2024 that will close this issue
@ghost ghost closed this as completed Feb 26, 2024
@epociask epociask reopened this Feb 26, 2024
@engr0001
Copy link

engr0001 commented Mar 3, 2024

It would be also great if we could alert certain events via Telegram

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants