Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(treewide): initial attempt at migrating to newer PTB #3

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

SphericalKat
Copy link

Signed-off-by: Amogh Lele [email protected]

@ajnsit
Copy link
Member

ajnsit commented Nov 19, 2022

@SphericalKat also it seems that there's some work going on inside the python-telegram-bot library (which is used by this bot) - python-telegram-bot/python-telegram-bot#3340

@SphericalKat
Copy link
Author

SphericalKat commented Nov 19, 2022

@SphericalKat also it seems that there's some work going on inside the python-telegram-bot library (which is used by this bot) - python-telegram-bot/python-telegram-bot#3340

Indeed! I'm using the library from that in-progress feature branch currently. (forgot to update and push the requirements.txt, doing it soon). For the most part though, the changes involved here are because the library has breaking changes, and we would need to make these changes anyway.

The actual fix for topics is adding few minor kwargs here and there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants