-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
feat: import new remote feature flag controller with override functionality #23487
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
Merged
asalsys
merged 40 commits into
main
from
use-updated-remote-feature-flag-to-handle-overrides
Dec 19, 2025
+1,086
−1,385
Merged
Changes from 1 commit
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
579e947
import local remote feature flag
asalsys 5587dfe
fix import issue and working remote-feature-flag-controller
asalsys 369d995
support ab-test options
asalsys caaafbf
remove engine ready
asalsys 06336f2
update abTestRawFlags to rawProcessedRemoteFeatureFlags
asalsys eee57f6
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
asalsys 4cdecbd
update the rawFeatureFlag options
asalsys 79726ad
use preiview remote-feature-flag-controller
asalsys 8b612df
enum flag type
asalsys 5c9cd1e
clean up context
asalsys 570dd64
fix override dashboard
asalsys 2283792
null state
asalsys 330c103
remove featureflag Description
asalsys 99f1c93
fix tsc
asalsys 0066cac
use remote-feature-flag-controller 3.1.0
asalsys 674cc59
override logic requires rerender
asalsys 022b5fc
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
asalsys c372a47
fix tests
asalsys 00f686d
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
asalsys 8152416
update initial background state
asalsys 081617c
update snapshots
asalsys c622a8b
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
asalsys b3d03c6
fix stat discrepency
asalsys f1fd362
update tests
asalsys 3a59724
return default display if ab test break
asalsys 51d4ce9
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
asalsys a94723a
update overrides on dashboard
asalsys ed788fb
fix error
asalsys 492ce47
fix test
asalsys 885935d
fix test
asalsys d4c707c
add test coverage
asalsys 60174f3
update tests
asalsys f2c27d7
add FeatureFlagBooleanNested case
asalsys 13544fa
ab test undefined issue
asalsys 236faec
fix FeatureFlagBooleanWithMinimumVersion
asalsys 0aa6981
diable ab test if raw flag values not loaded
asalsys 8caf5eb
for string inputs dont refresh while editing
asalsys a39e09c
handle ref gracefully
asalsys 75069a8
fix resetting ref issue
asalsys 04dea27
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
asalsys File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Merge branch 'main' into use-updated-remote-feature-flag-to-handle-ov…
…errides
- Loading branch information
commit 51d4ce99d83309ed0221a1ba69f6bb7ef9849af7
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
You are viewing a condensed version of this merge commit. You can view the full changes here.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 doesn't seem like it should do anything since it's triggering an empty function from inside of the useEffect. What's the intended purpose here - is it to get the latest state every time the feature flag changes?