-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Webhook] : update doc #19974
base: 6.4
Are you sure you want to change the base?
[Webhook] : update doc #19974
Conversation
af22c0a
to
ee47ae8
Compare
------------------------------------------------------------------ | ||
|
||
It is important to note that when the incoming webhook is processed by the :class:`WebhookController`, you have the option to handle the consumption of remote events asynchronously. | ||
Indeed, this can be configured using a bus, with the default setting pointing to the Messenger component's default bus. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
todo: show config
bb79ac4
to
581043f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just had time for a quick review of the first bit of this document.
I was a bit confused on "who was the consumer and who was the provider".
581043f
to
ea9732e
Compare
Update the webhook documentation