search

caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
10.94k stars 385 forks source link

[Telegram] `topic` should be associated with `chat_id` #1021

Closed gera2ld closed 6 months ago

gera2ld commented 6 months ago

:bulb: The Idea

When sending telegram messages, topic should be associated with chat_id. This could be a problem when specifying multiple chat_ids at the same time. Maybe it's better to change the URL into this format?

tgram://{bot_token}/{chat_id1}:{topic1}/{chat_id2}:{topic2}/{chat_id3}

:hammer: Breaking Feature

We can still use query.topic as a fallback, so it will not break anything.

caronc commented 6 months ago

That's a really cool idea! 🚀

caronc commented 6 months ago

Closing issue off as it has been merged