-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[Messenger] Add RunProcessMessage, RunCommandMessage and PingWebhookMessage
#18631
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
[Messenger] Add RunProcessMessage, RunCommandMessage and PingWebhookMessage
#18631
Conversation
8474e98 to
a4ff0c7
Compare
a4ff0c7 to
1d86bd5
Compare
RunProcessMessageRunProcessMessage, RunCommandMessage and PingWebhookMessage
1d86bd5 to
bb8239d
Compare
|
@OskarStark that's update 👍 |
bb8239d to
be38ac0
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.
Note similar to above that a ResponseInterface is returned from the handler?
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.
Added 👍
be38ac0 to
4b688bb
Compare
4b688bb to
7544680
Compare
kbond
left a comment
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.
👍🏻
7544680 to
91a358e
Compare
|
What a great contribution (fixing 3 issues!), what a nice feature and what an amazing review process. Thank you all for being so marvelous 🙇 |
RunProcessMessageand `RunProcessMessageHand… #18629PingWebhookMessageand `PingWebhookMessag… #18636RunCommandMessageand `RunCommandMessageHand… #18643