-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add event hooks to client #1
Add event hooks to client #1
Conversation
7d58127
to
1036a7f
Compare
* Add init hook, exposed through postmate model. Must be called by parent after handshake with app context data before all other operations may proceed. * Add exec hook, exposed through postmate model, for the parent to send events to the child. * Add `on` event handler subscription method, with subscribe and unsubscribe logic * Add checks in `exec` handler verifying that the user installation includes the capabilities necessary to recieve an event of the given type. If not, a debug error is logged and the subscribed handler is not executed.
1036a7f
to
6f09aaa
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.
looks great! The sandboxing is super helpful.
6351efd
to
6640de3
Compare
Ignore thisThere was an issue when setting up 🚀 PR was released in |
Ignore thisThere was an issue when setting up 🚀 PR was released in |
on
event handler subscription method, with subscribe and unsubscribe logicexec
handler verifying that the user installation includes the capabilities necessary to recieve an event of the given type. If not, a debug error is logged and the subscribed handler is not executed.