search

grammyjs / grammY

The Telegram Bot Framework.
https://grammy.dev
MIT License
2.36k stars 116 forks source link

feat: Add `bot.chosenInlineResult` #215

Closed askador closed 1 year ago

askador commented 2 years ago

this registers middleware for chosen_inline_result that will be executed for a certain result_id(s) 

bot.chosenInlineResult('resultId', async ctx => {...})
tusharwebd commented 2 years ago

Hi! I would like to work on this request. This is going to be the first open-source issue I'll work on. Please let me know if you can assign this task to me. Thanks!

KnorpelSenf commented 2 years ago

Hi! I would like to work on this request. This is going to be the first open-source issue I'll work on. Please let me know if you can assign this task to me. Thanks!

Great to hear, looking forward to your contribution!

KnorpelSenf commented 2 years ago

Feel free to ask any questions if you get stuck or need help, either here on GitHub or in https://t.me/grammyjs :)

KnorpelSenf commented 2 years ago

@tusharwebd did you find any time to look into this yet?

tusharwebd commented 2 years ago

@tusharwebd did you find any time to look into this yet?

Hi! Sorry for the delay. I've started working on this and will have an update on this by Monday?!

KnorpelSenf commented 2 years ago

No worries, there's no hurry. This change is supposed to land in 1.11 but we have the 1.10 release before that anyway, so please take your time. I was just curious if you were still interested in working on this, so thanks for responding :)

KnorpelSenf commented 2 years ago

The new update changed a lot about the internals that are relevant for this. Let me know if you have any questions about how to work on this :)

KnorpelSenf commented 2 years ago

@tusharwebd I assume you no longer feel like working on this, how is that? :(

MrVSiK commented 2 years ago

Hi! Is anyone working on this feature?

KnorpelSenf commented 2 years ago

It does not seem like that. @MrVSiK it would be great if you can take over, are you interested in this?

MrVSiK commented 2 years ago

Yes, I'd like to give it a shot