hacettepeoyt / hu-announcement-bot

Get the latest from Hacettepe with this amazing Telegram Bot!
https://t.me/HacettepeDuyurucuBot
GNU General Public License v3.0
8 stars 6 forks source link

Adding and removing back to back doesn't work #58

Closed furkansimsekli closed 7 months ago

furkansimsekli commented 7 months ago

Try this:

  1. /add
  2. Choose dep_x
  3. /remove
  4. Choose dep_y

Expected: dep_x should be added to list of subscribed departments and dep_y should be removed from the list.

Output: dep_y is being removed and added back into the list immediately.

It doesn't matter whether dep_x and dep_y are the same departments. Issue also occurs when you do first /remove, and then /add.

What's wrong here?

I believe changing the conversation doesn't kill the old conversation. I thought I have implemented this way when I first introduced ConversationHandler for both of those commands, but apparently it doesn't work. This can be seen when you send /done in the 4th step of the example given above. It says Alright, let me know if you need anything else ;) twice.