hubot-telegram icon indicating copy to clipboard operation
hubot-telegram copied to clipboard

inline query support

Open freeseacher opened this issue 9 years ago • 9 comments

according to latest api changes please add inline query support to your api https://core.telegram.org/bots/inline tnx!

freeseacher avatar Jan 05 '16 10:01 freeseacher

Yepp...on my to do list :) maybe today or tomorrow

On Tuesday, January 5, 2016, Aleksey Shirokih [email protected] wrote:

according to latest api changes please add inline query support to your api https://coretelegramorg/bots/inline tnx!

— Reply to this email directly or view it on GitHub https://github.com/lukefx/hubot-telegram/issues/32.

lukefx avatar Jan 05 '16 10:01 lukefx

Let me know if you need help on this @lukefx, might need to do some refactoring to support it with minimal code duplication.

arcturial avatar Jan 06 '16 08:01 arcturial

@arcturial yepp thx, first of all, what do you think for the inlineQuery initial request? https://core.telegram.org/bots/api#inlinequery we should manage the request within the telegrambot api and launch and event to the adapter? What do you think?

lukefx avatar Jan 06 '16 11:01 lukefx

All the telegrambot API does is interact with the getUpdate method and sends the result to the callback, so not sure it would be the best place to put the events. However, we can change the code at line 212 and 225 to call handleInlineQuery if the inline_query key is set and then abstracting all the "common tasks" into different methods.

arcturial avatar Jan 06 '16 11:01 arcturial

All the telegrambot API does is interact with the getUpdate method and sends the result to the callback, so not sure it would be the best place to put the events. However, we can change the code at line 212 and 225 to call handleInlineQuery if the inline_query key is set and then abstracting all the "common tasks" into different methods.

I was thinking a method to let user define what the bot should reply to the inlinquery, so yes, quick and dirty could be in the handleUpdate method add an if and check if the message starts with a "@" and then? send an event so you can catch it in a script and compose your reply...

Anyway, let's create a branch...and collaborate on it...code speak itself :smile:

lukefx avatar Jan 06 '16 11:01 lukefx

the text will not starts with @ i had already seen it. instead of it you will have the inline_query object with text in it

ср, 6 янв. 2016 г. в 16:45, Luca Simone [email protected]:

All the telegrambot API does is interact with the getUpdate method and sends the result to the callback, so not sure it would be the best place to put the events. However, we can change the code at line 212 and 225 to call handleInlineQuery if the inline_query key is set and then abstracting all the "common tasks" into different methods.

I was thinking a method to let user define what the bot should reply to the inlinquery, so yes, quick and dirty could be in the handleUpdate method add an if and check if the message starts with a "@" and then? send an event so you can catch it in a script and compose your reply...

Anyway, let's create a branch...and collaborate on it...code speak itself [image: :smile:]

— Reply to this email directly or view it on GitHub https://github.com/lukefx/hubot-telegram/issues/32#issuecomment-169305868 .

freeseacher avatar Jan 06 '16 11:01 freeseacher

Ah ok...so how do we know if is an inline query? By the object's attributes?

#33 I've created a branch for this...

lukefx avatar Jan 06 '16 11:01 lukefx

Teap, the object attributes we get in getUpdates are different. Like @freeseacher said, it will contain the inline_query key with the text of the command inside inline_query.query.

arcturial avatar Jan 06 '16 20:01 arcturial

any news on this? Would love to see that feature.

C3realGuy avatar Sep 23 '16 11:09 C3realGuy