bluewave-labs
Telegram doesn't seem to be an available notification type?
Is your feature request related to a problem? Please describe. When I look through the documentation and some of the code, there are references to Telegram support like here and here.
But when I got to the /notifications section of the app, I only see these notification types:
https://github.com/bluewave-labs/Checkmate/blob/develop/client/src/Pages/Notifications/utils.js#L1
Have I misunderstood something?
Describe the solution you'd like
Telegram should be available on the dropdown on the /notifications page.
Describe alternatives you've considered It seems like there's some other kind of notification adding system I haven't seen, or is it a paid feature?
Thank you!
Hello @Fluxanoia - we have removed that from the list as it was misbehaving. We can add it back and I'll leave this github issue open for now to see if someone is interested in taking it!
Ah okay, thank you for clarifying! Is there a related issue for the misbehaviour?
Ah okay, thank you for clarifying! Is there a related issue for the misbehaviour?
Currently no, it was a last minute cut :-D
Hey i solved the issue can you check my PR
Can you close your PR and reopen again and make sure you follow PR template/rules?
Hi, can you assign this issue to me if no one is working on this issue currently.
Thanks. Please go ahead. Make sure you follow PR template when opening the PR..
Are we able to get some clarification on what the issue was with the feature before it was cut? Just so we can keep an eye out for it?
Are we able to get some clarification on what the issue was with the feature before it was cut? Just so we can keep an eye out for it?
Hey @Fluxanoia ,
There isn't any issue with telegram notifications really, we just haven't implemented it yet.
Telegram webhooks require a token, so they must be handled in a different way than the regular webhooks like Slack, Discord, or generic webhooks.
It's not a huge deal to implement it, but it does need to be done.
Ah okay, thanks! I just misunderstood then, sounds good.
I see it's already assigned but I would be happy to pick this up if there isn't any movement in a week or so.
Ah okay, thanks! I just misunderstood then, sounds good.
I see it's already assigned but I would be happy to pick this up if there isn't any movement in a week or so.
👍
For telegram I managed creating a webhook on my n8n instance and put that URL as a webhook notification on checkmate
I am already working on the issue. Is it ok if I complete it in 4-5 days?
I am already working on the issue. Is it ok if I complete it in 4-5 days?
Sure, let us know :)
For integrating telegram notification, you will need a bot, so we have to let users create their own bot, save the token there and also the chat ID.
I can work on what @a-y-a-n-das has suggested, Assign this issue to me if no one is working on this currently.
Just start working, don't wait for the assignment, you can check my repo branch "added-telegram-notification".
Nope, the telegram wasn't showing in the dropdown; some UI got broken, some other things also got broken, so I just left it. You can try. Best of luck.
Hey, it looks like there hasn't been any progress on this, hence I'm working on fixing this using the following steps:
Attempting to set up a Telegram webhook via ngrok, so I can leverage the existing webhook functionality.
If that doesn't work, I'll move forward with implementing it on the backend directly.
Will keep you posted on the progress. Let me know if there are any suggestions
Ah okay, thank you for clarifying! Is there a related issue for the misbehaviour?
Currently no, it was a last minute cut :-D
When was it cut? Easier to restore the "broken" one and fix it instead of re-implemented front and backend
@darth-divyansh are you still working on this? I could give it a try, but don't want to waste time if you're almost finished.
