From 87cfcd9baee2a28dee53842e6fea79bc5dea9fd3 Mon Sep 17 00:00:00 2001 From: Kyle Johnson Date: Wed, 28 Feb 2024 13:15:14 +0000 Subject: [PATCH] fix: toggle flag (#3480) --- .../common/providers/FeatureListProvider.js | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/frontend/common/providers/FeatureListProvider.js b/frontend/common/providers/FeatureListProvider.js index 297d4e73c2d6..0de8ab917626 100644 --- a/frontend/common/providers/FeatureListProvider.js +++ b/frontend/common/providers/FeatureListProvider.js @@ -51,15 +51,26 @@ const FeatureListProvider = class extends React.Component { this.editFeatureValue( projectId, environmentId, + /* todo: Saving features involves sending an adjusted project flag rather than a feature state (old tech debt). + This will be removed when migrating to RTK. The following converts the feature state to the accepted format. + */ { ...projectFlag, default_enabled: !environmentFlag.enabled, + initial_value: environmentFlag.feature_state_value, + multivariate_options: projectFlag.multivariate_options.map((mv) => { + const matching = + environmentFlag.multivariate_feature_state_values.find( + (v) => v.multivariate_feature_option == mv.id, + ) + return { + ...mv, + default_percentage_allocation: matching.percentage_allocation, + } + }), }, projectFlag, - { - ...environmentFlag, - enabled: !environmentFlag.enabled, - }, + environmentFlag, ) }