search

leam-tech / frappe_telegram

Telegram Bot support for Frappe / ERPNext.
Other
32 stars 24 forks source link

feat: Support for sending files #4

Closed Abadulrehman closed 3 years ago

assemmarwan commented 3 years ago

@Abadulrehman What do you think about having another function where you can send file + message?

Abadulrehman commented 3 years ago

@Abadulrehman What do you think about having another function where you can send file + message?

@assemmarwan If you mean a message that shows up together with the file, I think that is impossible because Telegram does not support sending files + message together. If you mean sending a file and then a message, then can't we just use "send_message" followed by "send_file"?

assemmarwan commented 3 years ago

@Abadulrehman What do you think about having another function where you can send file + message?

@assemmarwan If you mean a message that shows up together with the file, I think that is impossible because Telegram does not support sending files + message together. If you mean sending a file and then a message, then can't we just use "send_message" followed by "send_file"?

Yes that's what I meant a function that calls these two 😅.

Abadulrehman commented 3 years ago

@Abadulrehman What do you think about having another function where you can send file + message?

@assemmarwan If you mean a message that shows up together with the file, I think that is impossible because Telegram does not support sending files + message together. If you mean sending a file and then a message, then can't we just use "send_message" followed by "send_file"?

Yes that's what I meant a function that calls these two sweat_smile.

@assemmarwan If you mean a message that shows up together with the file, I think that is impossible because Telegram does not support sending files + message together.

@assemmarwan My mistake, it actually does. Added it to our wrapper.