-
Notifications
You must be signed in to change notification settings - Fork 6k
Closed
Labels
⚙️ bot-apiaffected functionality: bot-apiaffected functionality: bot-api
Milestone
Description
I just noticed that ptbs telegram.ReplyMarkup is in fact an invention of PTB, i.e. it's not defined by the official API docs. We should consider removing that class or at least making it private.
IISC it's currently only used for
- type hinting - that could easily be replaced by a type alias
- the static method
_check_keyboard_typethat's used in the inits of or{Reply, Inline}KeyboardMarkup- that could moved e.g. totg._utils
Personally, I'd be in favor of just removing the class. @harshil21 IIRC you added the _check_keyboard_type method - what do you think?
Metadata
Metadata
Assignees
Labels
⚙️ bot-apiaffected functionality: bot-apiaffected functionality: bot-api