Salz0 / telegram_flea

A open-source flea market bot, to be used in any Community
MIT License
9 stars 21 forks source link

Let the person know which request was canceled #52

Open Salz0 opened 10 months ago

Salz0 commented 10 months ago

What's happening?

Currently the bot simply types the cancelation message without any reference to the request

Why?

It should be more intuitive to do a bot.reply() instead of bot.send_message() because the person will understand what is happening

This is easy to fix

sfazli96 commented 10 months ago

I just want to make sure the fix should be bot.reply() where the bot replies instead of it sending a message? So the fix is I would change it in the file somewhere, correct? I can give this issue a try.

mykolasolodukha commented 10 months ago

@Salz0 bot.reply() implies that we already have message_id somewhere saved (either locally or in the database). Is that the case or is it irrelevant?

mykolasolodukha commented 10 months ago

@sfazli96 feel free to give it a go. If anything, we're here to help 🙂

Salz0 commented 10 months ago

@Salz0 bot.reply() implies that we already have message_id somewhere saved (either locally or in the database). Is that the case or is it irrelevant?

The cancelation button should include the message_id. That way it should be good

Salz0 commented 10 months ago

@Salz0 bot.reply() implies that we already have message_id somewhere saved (either locally or in the database). Is that the case or is it irrelevant?

The cancelation button should include the message_id. That way it should be good

@sfazli96 @mykolasolodukha