Flags.enable and FlagsClient APIs#2900
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## feature/feature-flagging #2900 +/- ##
============================================================
+ Coverage 70.76% 70.86% +0.10%
============================================================
Files 834 834
Lines 30340 30381 +41
Branches 5128 5131 +3
============================================================
+ Hits 21469 21529 +60
- Misses 7415 7421 +6
+ Partials 1456 1431 -25
🚀 New features to boost your workflow:
|
…ent-manager-or-eliminate' into typo/FFL-1117-pass-default-client-configuration-to-flags-enable
…ent-manager-or-eliminate' into typo/FFL-1117-pass-default-client-configuration-to-flags-enable
…ent-manager-or-eliminate' into typo/FFL-1117-pass-default-client-configuration-to-flags-enable
0xnm
left a comment
There was a problem hiding this comment.
Resetting review status, because there is file which shouldn't be probably committed.
0349f3e to
cdebda3
Compare
typotter
left a comment
There was a problem hiding this comment.
alas, I have not been able to quite get through the merge conflicts from Jonathan's merges. tomorrow!
…lt-client-configuration-to-flags-enable
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
Flags Feature: Enable method
Feature Flagging mobile SDK MVP spike
stacked on #2899
FFL-1117
FFL-1118
What does this PR do?
This PR implements the complete Flags feature initialization and client management APIs, bringing both
Flags.enable()andFlagsClientup to the mobile API specification. The PR includes configuration management, multi-domain client support, and graceful degradation patterns.Key Changes:
1. Flags.enable() Configuration
FlagsConfigurationwithcustomExposureEndpointFlagsFeatureto accept and storeFlagsConfigurationfor future client creationFlagsContextand network layer2. FlagsClient Factory API
get()method with multi-tenant support:get(name: String = "default", sdkCore: SdkCore)setContext()tosetEvaluationContext()for OpenFeature alignment3. Multi-Client Management
unregister()andclear()methodsMotivation
This change fully implements the mobile API specification for Feature Flagging:
Review checklist (to be filled by reviewers)