-
Notifications
You must be signed in to change notification settings - Fork 22
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: Statsing provider #163
Conversation
Signed-off-by: Jens Henneberg <jens.henneberg@phocassoftware.com>
Signed-off-by: Jens Henneberg <jens.henneberg@phocassoftware.com>
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.
Overall, it looks good. I left a few comments but I think we can merge this when you're ready.
src/OpenFeature.Contrib.Providers.Statsig/EvaluationContextExtensions.cs
Outdated
Show resolved
Hide resolved
Thanks for having a look. 😄 I'll do some more testing and wait for a decision on a new release of https://github.com/open-feature/dotnet-sdk/ before merging. |
Signed-off-by: Jens Henneberg <jens.henneberg@phocassoftware.com>
Updated to |
Signed-off-by: Jens Henneberg <jens.henneberg@phocassoftware.com>
This PR
Notes
The implementation is minimal - only
ResolveBooleanValue
is implemented.How to test
Create a free Statsig account and use as described in README.md