Skip to content
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

init version #3

Merged
merged 44 commits into from
Jul 3, 2024
Merged

init version #3

merged 44 commits into from
Jul 3, 2024

Conversation

chylli-deriv
Copy link
Contributor

@chylli-deriv chylli-deriv commented Jun 21, 2024

We are looking at creating simple SDK for Growthbook (Feature Flags).   We have written simple code to access flags but it will be great to get separate modules which can be used by multiple teams. here is the code we have 

@chylli-deriv chylli-deriv changed the title Add feature init version Jun 26, 2024
t/growthbook.t Outdated Show resolved Hide resolved
examples/feature.pl Outdated Show resolved Hide resolved
examples/feature.pl Outdated Show resolved Hide resolved
field $client_key :param;
field $features :param //= {};
field $cache_ttl :param //= 60;
field $cache //= WebService::GrowthBook::InMemoryFeatureCache->singleton;

Choose a reason for hiding this comment

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

do we need this cache object here ? From my understanding The FeatureRepository seems to be abstracting the cache from the the Growthbook class Object ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This parameter is used to set cache module in FeatureRepository. Without this one we cannot custom cache object in FeatureRepository.

  - Initial release
  - load features
  - get feature value
  - is_on
  - is_off
@mukesh-deriv mukesh-deriv merged commit 2563883 into deriv-com:master Jul 3, 2024
6 checks passed
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.

5 participants