Different handlers for different bots?

[intoeetive]

Hi @fabio-ivona

First of all - this is an excellent piece of software! I am writing telegram bot for a volunteer project, and it's really speeding up the development!

Is it possible use different Webhook Handlers for different bots? So that in config file instead of
'webhook_handler' => MyWebhookHandler::class,
I could have something like:

'webhook_handlers' => [
    'user_bot' => MyWebhookHandler::class,
    'admin_bot' => MyAdminWebhookHandler::class,

I think this might be a lot more convenient then figuring out where the request came from in handler itself

Also been wondering if you have plans to support request_contact button anytime soon? (if not, I might submit a PR for that in next few days)

[fabio-ivona]

Hi! @intoeetive !

thanks for your kind words, we are glad you find Telegraph useful

Dedicated webhook handlers are not currently supported, but you could achieve something similar by defining a main webhook handler like this one:

class MainWebhookHandler extends WebhookHandler
{
   public function handle(Request $request, TelegraphBot $bot): void
   {
        match($bot->name){
            'user_bot' => app(MyWebhookHandler::class)->handle($request, $bot),
            'admin_bot' => app(MyAdminWebhookHandler::class)->handle($request, $bot),
            default => throw new Exception('unsupported bot');
       }
   }
}

[intoeetive]

Oh that's handy, did not think of that approach. Thank you!

[ah-rahimi]

@fabio-ivona
Excuse me
In MyAdminWebhookHandler class & MyWebhookHandler class we must extends WebHookHandler ?

[fabio-ivona]

Yeah, so it will inherit all its methods

[fabio-ivona]

Also been wondering if you have plans to support request_contact button anytime soon? (if not, I might submit a PR for that in next few days)

we currently support only inline keyboards, but I am working on reply keyboards and they will be released in a few days. As soon as it is ready your PR for request_contact is welcome! thanks!

[fabio-ivona]

Hi @intoeetive I think we will merge it on master branch in a couple of days (so you can get it by requiring dev-master as beta version) and will release a stable version by the end of the week

[intoeetive]

@fabio-ivona i don't see that branch on GitHub, is there any chance it's not pushed?

[fabio-ivona]

Not yet pushed, we are refining it and it will be available on a couple of days. Will post an update as soon as it is available

[fabio-ivona]

@intoeetive #94 😉

you can require this version for an early test:

composer require defstudio/telegraph:dev-reply-keyboards

let me know if it works!

[intoeetive]

Thank you - pulling it in works, I'll give actual test today or tomorrow