rumdood / Moneo

Telegram Bot for Managing Scheduled Tasks and Reminders
MIT License
2 stars 0 forks source link

Task Name for Commands Should Not Be Case-Sensitive #15

Open rumdood opened 9 months ago

rumdood commented 9 months ago

When a task is created from the chatbot commands (i.e. /create), the casing used is what gets echoed back to the user, but the API's GET calls are case-sensitive and tasks are created in all-lower-case.

Scenario:

User: /create MyTask ...[Q&A to build task] Bot: Created MyTask! User: /complete MyTask <--- Backend receives a 404 error because the task MyTask is not found by the API User: /complete mytask <--- Succeeds

rumdood commented 9 months ago

There are actually two issues here. One in that the task can't be complete unless the user sends the task name in lower case and the other that the bot does not actually respond when it receives the 404 error. No feedback is given to the user at all.