-
Notifications
You must be signed in to change notification settings - Fork 19
feat: Add support for hooks. #220
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
Conversation
bugbot review |
…arkly/flutter-client-sdk into rlamb/o11y-450/add-hooks-support
bugbot review |
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.
Updating these tests for environment ID now that it has been added.
} | ||
|
||
Future<IdentifyResult> _mapIdentifyResult(TaskResult<void> res) async { | ||
IdentifyResult _mapIdentifyResult(TaskResult<void> res) { |
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.
This method didn't need to be async as it runs only on already resolved promises.
🤖 I have created a release *beep* *boop* --- ## [1.7.0](launchdarkly_common_client-v1.6.2...launchdarkly_common_client-v1.7.0) (2025-09-12) ### Features * Add experimental plugin support. ([#225](#225)) ([5bd9ce7](5bd9ce7)) * Add support for hooks. ([#220](#220)) ([6e7a26d](6e7a26d)) * Internal environment ID support. ([#217](#217)) ([71b522b](71b522b)) ### Bug Fixes * Change hook data values to `dynamic` from `LDValue`. ([d7720f3](d7720f3)) * Export required plugin meta-data types. ([d7720f3](d7720f3)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- ## [4.12.0](4.11.2...4.12.0) (2025-09-15) ### Features * Add experimental plugin support. ([#225](#225)) ([5bd9ce7](5bd9ce7)) * Add hook support and experimental plugin support. ([#228](#228)) ([b698905](b698905)) * Add support for hooks. ([#220](#220)) ([6e7a26d](6e7a26d)) * Update version constraints for device_info_plus to allow using 12.x. ([9b5f530](9b5f530)) * Update version constraints for package_info_plus to allow using 9.x. ([9b5f530](9b5f530)) ### Bug Fixes * Change hook data values to `dynamic` from `LDValue`. ([d7720f3](d7720f3)) * Export required plugin meta-data types. ([d7720f3](d7720f3)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: Ryan Lamb <[email protected]>
Add support for hooks to the SDK.
Example of a hook with all methods that logs things.
Minimal hook: