diff options
Diffstat (limited to 'app/assets/javascripts/feature_flags/components/feature_flags_table.vue')
-rw-r--r-- | app/assets/javascripts/feature_flags/components/feature_flags_table.vue | 68 |
1 files changed, 9 insertions, 59 deletions
diff --git a/app/assets/javascripts/feature_flags/components/feature_flags_table.vue b/app/assets/javascripts/feature_flags/components/feature_flags_table.vue index 9220077af71..cfd838bf5a1 100644 --- a/app/assets/javascripts/feature_flags/components/feature_flags_table.vue +++ b/app/assets/javascripts/feature_flags/components/feature_flags_table.vue @@ -1,8 +1,7 @@ <script> -import { GlBadge, GlButton, GlTooltipDirective, GlModal, GlToggle, GlIcon } from '@gitlab/ui'; +import { GlBadge, GlButton, GlTooltipDirective, GlModal, GlToggle } from '@gitlab/ui'; import { __, s__, sprintf } from '~/locale'; import glFeatureFlagMixin from '~/vue_shared/mixins/gl_feature_flags_mixin'; -import { ROLLOUT_STRATEGY_PERCENT_ROLLOUT, NEW_VERSION_FLAG, LEGACY_FLAG } from '../constants'; import { labelForStrategy } from '../utils'; export default { @@ -14,7 +13,6 @@ export default { components: { GlBadge, GlButton, - GlIcon, GlModal, GlToggle, }, @@ -35,13 +33,7 @@ export default { deleteFeatureFlagName: null, }; }, - translations: { - legacyFlagReadOnlyAlert: s__('FeatureFlags|Flag is read-only'), - }, computed: { - permissions() { - return this.glFeatures.featureFlagPermissions; - }, modalTitle() { return sprintf(s__('FeatureFlags|Delete %{name}?'), { name: this.deleteFeatureFlagName, @@ -57,12 +49,6 @@ export default { }, }, methods: { - isLegacyFlag(flag) { - return flag.version !== NEW_VERSION_FLAG; - }, - statusToggleDisabled(flag) { - return flag.version === LEGACY_FLAG; - }, scopeTooltipText(scope) { return !scope.active ? sprintf(s__('FeatureFlags|Inactive flag for %{scope}'), { @@ -70,22 +56,6 @@ export default { }) : ''; }, - badgeText(scope) { - const displayName = - scope.environmentScope === '*' - ? s__('FeatureFlags|* (All environments)') - : scope.environmentScope; - - const displayPercentage = - scope.rolloutStrategy === ROLLOUT_STRATEGY_PERCENT_ROLLOUT - ? `: ${scope.rolloutPercentage}%` - : ''; - - return `${displayName}${displayPercentage}`; - }, - badgeVariant(scope) { - return scope.active ? 'info' : 'muted'; - }, strategyBadgeText(strategy) { return labelForStrategy(strategy); }, @@ -142,7 +112,6 @@ export default { <gl-toggle v-if="featureFlag.update_path" :value="featureFlag.active" - :disabled="statusToggleDisabled(featureFlag)" :label="$options.i18n.toggleLabel" label-position="hidden" data-testid="feature-flag-status-toggle" @@ -169,12 +138,6 @@ export default { <div class="feature-flag-name text-monospace text-truncate"> {{ featureFlag.name }} </div> - <gl-icon - v-if="isLegacyFlag(featureFlag)" - v-gl-tooltip.hover="$options.translations.legacyFlagReadOnlyAlert" - class="gl-ml-3" - name="information-o" - /> </div> <div class="feature-flag-description text-secondary text-truncate"> {{ featureFlag.description }} @@ -189,27 +152,14 @@ export default { <div class="table-mobile-content d-flex flex-wrap justify-content-end justify-content-md-start js-feature-flag-environments" > - <template v-if="isLegacyFlag(featureFlag)"> - <gl-badge - v-for="scope in featureFlag.scopes" - :key="scope.id" - v-gl-tooltip.hover="scopeTooltipText(scope)" - :variant="badgeVariant(scope)" - :data-qa-selector="`feature-flag-scope-${badgeVariant(scope)}-badge`" - class="gl-mr-3 gl-mt-2" - >{{ badgeText(scope) }}</gl-badge - > - </template> - <template v-else> - <gl-badge - v-for="strategy in featureFlag.strategies" - :key="strategy.id" - data-testid="strategy-badge" - variant="info" - class="gl-mr-3 gl-mt-2 gl-white-space-normal gl-text-left gl-px-5" - >{{ strategyBadgeText(strategy) }}</gl-badge - > - </template> + <gl-badge + v-for="strategy in featureFlag.strategies" + :key="strategy.id" + data-testid="strategy-badge" + variant="info" + class="gl-mr-3 gl-mt-2 gl-white-space-normal gl-text-left gl-px-5" + >{{ strategyBadgeText(strategy) }}</gl-badge + > </div> </div> |