rougeth / bottery

[DEVELOPMENT-HALTED] :battery: A bot framework with batteries included
MIT License
344 stars 51 forks source link

#179: Added general telegram exceptions handling. #180

Closed ivkalita closed 11 months ago

ivkalita commented 6 years ago

I added asyncio loop exception handler and Telegram exception handling on top of it.

Closes: #179

rougeth commented 6 years ago

Hi @kaduev13, thanks for the PR. I made some comments, could you take a look at them? Also, what do you think about adding some tests?

ivkalita commented 6 years ago

Hi @rougeth! Thank you for the review, I'll make the fixes tomorrow. About the tests – yeah, I'll add them, sorry for not doing it in the initial PR – I tried to find polling test case but found nothing and decided to send you a draft without tests to discuss it first.