Telegram module #10

Merged
skobkin merged 19 commits from feature_telegram into master 2022-05-30 23:54:27 +03:00
Collaborator

telegram module. Closes #2 #7.

Two service classes:

  • Notifier class
  • CommandProcessor class

Two middlewares:

  • UserAuthMiddleware
  • ExceptionHandlerMiddleware

One exception for usage in the code called by the bot:

  • DisplayableException

Merge ONLY after #9 is merged.

`telegram` module. Closes #2 #7. Two service classes: * `Notifier` class * `CommandProcessor` class Two [middlewares](https://github.com/eternnoir/pyTelegramBotAPI/tree/master/examples/middleware): * `UserAuthMiddleware` * `ExceptionHandlerMiddleware` One exception for usage in the code called by the bot: * `DisplayableException` Merge **ONLY** after #9 is merged.
skobkin added this to the MVP 0.1 milestone 2022-05-02 02:45:43 +03:00
skobkin self-assigned this 2022-05-02 02:45:43 +03:00
skobkin force-pushed feature_telegram from 01dda4fafe to aa75492716
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
2022-05-02 15:55:08 +03:00
Compare
skobkin force-pushed feature_telegram from aa75492716
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
to eba2ab2252
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
2022-05-02 17:55:45 +03:00
Compare
skobkin force-pushed feature_telegram from a3a38a856e
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
to d39e046af2
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
2022-05-02 18:27:55 +03:00
Compare
skobkin force-pushed feature_telegram from 6c83447eb8
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
to ec6d707a56
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
2022-05-02 23:45:48 +03:00
Compare
skobkin force-pushed feature_telegram from ec6d707a56
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
to 74bd517238
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
2022-05-30 00:34:00 +03:00
Compare
skobkin force-pushed feature_telegram from 74bd517238
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
to 305f1afa3c
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
2022-05-30 00:36:34 +03:00
Compare
skobkin force-pushed feature_telegram from 9e4488b204
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
to 3032c642be
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
2022-05-30 03:24:09 +03:00
Compare
skobkin force-pushed feature_telegram from 3032c642be
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
to 882b4df5e1
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
2022-05-30 03:33:07 +03:00
Compare
skobkin changed title from WIP: Telegram package to WIP: Telegram module 2022-05-30 03:34:30 +03:00
#7 Removing unnecessary statement.
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
af73458e29
skobkin changed title from WIP: Telegram module to Telegram module 2022-05-30 03:35:27 +03:00
Author
Collaborator

@Miroslavsckaya This PR is more or less ready. Check it out when you have time.

@Miroslavsckaya This PR is more or less ready. Check it out when you have time.
@ -11,2 +10,3 @@
"""Create a database file if not exists."""
self.conn = sqlite3.connect(path)
# TODO: think about removing check_same_thread=False
self.conn = sqlite3.connect(path, check_same_thread=False)
Author
Collaborator

check_same_thread=False is a bad solution which can backfire on us in the future.

But I didn't see any other good solution at the time.

`check_same_thread=False` is a bad solution which can backfire on us in the future. But I didn't see any other good solution at the time.
#7 Notifier now sends messages in HTML.
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
3bffd32df0
#7 Notifier is now considering Telegram API sendMessage() rate limit.
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
f9ffdac38a
#7 Notifier counter reset fix.
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
8554689fef
#7 Notifier inconsistent naming fix.
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing
54cdb3f368
skobkin merged commit 2e44df0bbd into master 2022-05-30 23:54:27 +03:00
skobkin deleted branch feature_telegram 2022-05-30 23:54:27 +03:00
skobkin referenced this pull request from a commit 2022-05-30 23:54:27 +03:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#2 Telegram API integration
Miroslavsckaya/tg_rss_bot
Depends on
#9 Python venv
Miroslavsckaya/tg_rss_bot
Reference
Miroslavsckaya/tg_rss_bot!10
No description provided.