-
Notifications
You must be signed in to change notification settings - Fork 513
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
feat: onConnect w/ controller #3028
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3028 +/- ##
==========================================
- Coverage 93.56% 93.55% -0.02%
==========================================
Files 89 89
Lines 24301 24424 +123
==========================================
+ Hits 22737 22849 +112
- Misses 1564 1575 +11 ☔ View full report in Codecov by Sentry. |
1c3cdbf
to
61cfc20
Compare
Part of a larger overhauld of hooks. This on is not breaking and can therefore land earlier.
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.
LGTM, shall we document it and add types for it?
91b0f26
to
3d331a2
Compare
Updated according to feedback from @BridgeAR |
dc978ac
to
41dc118
Compare
ab0766f
to
714d50f
Compare
Closing in favour of #3051 |
Part of a larger overhauld of hooks. This on is not breaking and can therefore land earlier.
This relates to...
Rationale
Changes
Features
Bug Fixes
Breaking Changes and Deprecations
Status