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
Refactor Event Handling #47
Conversation
rename EventManager => Events expose useEventHandler
Hi @prevwong will you include #35 and #37 in your refactor solution? Then I'll close them in favor of your solution here. Otherwise I'll try to find some time asap to merge #35 - on #37 I would need your feedback to continue. If you're not in a hurry - would now be a good time to add some tests? Maybe some cypress integration tests? |
@ankri Let's keep those in a separate PR. Yeah, will be adding tests with Cypress but I will need some time for that. |
This PR is intended to Improve on the overall structure of how events are handled in Craft.
useHandlerGuard
anduseConnectorHooks
.DerivedEventHandlers
which makes it easier to bootstrap new handlers/connectors.