Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

pavkam
Copy link
Contributor

@pavkam pavkam commented Feb 8, 2025

This PR adds the ability to pass the meta to getFeature(s) and the bound client.

- Enhanced JSDoc comments for BucketClient methods
- Added more detailed parameter descriptions
- Improved type checking for meta context
- Extracted meta validation into a separate function
- Updated type definitions for context and tracking options
- Refined error messages for validation
@pavkam pavkam self-assigned this Feb 8, 2025
Copy link
Contributor

@roncohen roncohen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few nits, but otherwise LGTM!

- Reverted package version from 1.5.3 to 1.5.2
- Enhanced JSDoc comments for BucketClient constructor
- Added documentation for `secretKey` parameter
- Marked BoundBucketClient constructor as internal
@pavkam pavkam enabled auto-merge (squash) February 9, 2025 19:19
@pavkam pavkam merged commit de93d76 into main Feb 9, 2025
5 checks passed
@pavkam pavkam deleted the buc-3276-nodesdk-in-getfeatures-we-should-default-to-active-true-but branch February 9, 2025 19:23
roncohen pushed a commit that referenced this pull request Feb 26, 2025
This PR adds the ability to pass the `meta` to `getFeature(s)` and the
bound client.
roncohen pushed a commit that referenced this pull request Feb 26, 2025
This PR adds the ability to pass the `meta` to `getFeature(s)` and the
bound client.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants