Taragolis commented on PR #28953: URL: https://github.com/apache/airflow/pull/28953#issuecomment-1398463533
Well... my personal thoughts that there is not a lot of differences for end users between migration to `python-telegram-bot>=20.0.0` and replace by `pyTelegramBotAPI`. I think we could keep work Hooks/Operators as it work now in both cases. Breaking compatibility is new object returned by `TelegramHook.connection` and `TelegramHook.get_conn()` witch incompatible with current version in both cases. The main differences how end user who build their Operators will resolve incompatibility issues. 1. `python-telegram-bot>=20.0.0`: In most cases required to wrap all `telegram.Bot` async methods in sync implementation. 2. `pyTelegramBotAPI`: Need migrate to `telebot.TeleBot` methods Unfortunetly I'm not an enduser of Telegram Provider so I'm not a best person who could which approach would be less harmful for end users. @eladkal, @uranusjr Maybe you have you thoughts about this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
