search

argoproj / notifications-engine

Affordable notifications for Kubernetes
Apache License 2.0
271 stars 147 forks source link

Add option for Telegram to use a thread id when posting notification #265

Closed Techwolf12 closed 7 months ago

Techwolf12 commented 8 months ago

This also migrates the go library to the one from OvyFlash, since the current one seems to be no longer maintained: https://github.com/go-telegram-bot-api/telegram-bot-api/issues/695

codecov[bot] commented 8 months ago

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (ace379e) 55.14% compared to head (4479ff4) 55.50%.

Files Patch % Lines
pkg/services/telegram.go 42.85% 15 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #265 +/- ## ========================================== + Coverage 55.14% 55.50% +0.36% ========================================== Files 35 35 Lines 3400 3412 +12 ========================================== + Hits 1875 1894 +19 + Misses 1251 1242 -9 - Partials 274 276 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.