search

sonaalPradeep / arxiv-telegram-bot

0 stars 1 forks source link

Conversation handler fix #13

Open reuben-roy opened 2 years ago

reuben-roy commented 2 years ago

Conversation handler has issues which must be rectified to prevent crashes

https://github.com/sonaalPradeep/arxiv-telegram-bot/blob/ad73b27d03cbba1975e912a7ee285e0d8cc8bdf4/arxiv_telegram_bot/base.py#L233

reuben-roy commented 2 years ago

"/commands" must be filtered out Must filter out invalid /preferences Must add test cases

sonaalPradeep commented 2 years ago

Telegram bot module does provide filters for these specific cases (especially for commands)

sonaalPradeep commented 2 years ago

I think now since we are using inline keyboards, then the user won't be able to run commands and they'll be restricted to that conversation "box". Still see if you can break and try to fix it / raise issues for it