Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-05-03 01:18:43 +02:00
Code Issues Projects Releases Wiki Activity

chore: featureToggleService use optionallyDisableFeature instead of duplicating logic (#9073)

Browse Source
This commit is contained in:
David Leek 2025-01-09 11:29:00 +01:00 committed by GitHub
parent b0434c9a84
commit 0dbecd78a0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/lib/features/feature-toggle/feature-toggle-service.ts
View File

@ -983,28 +983,14 @@ class FeatureToggleService {
await this.featureStrategiesStore.delete(id); await this.featureStrategiesStore.delete(id);
const featureStrategies = // Disable the feature in the environment if it only has disabled strategies
await this.featureStrategiesStore.getStrategiesForFeatureEnv( await this.optionallyDisableFeature(
projectId, featureName,
featureName, environment,
environment, projectId,
); auditUser,
const hasOnlyDisabledStrategies = featureStrategies.every(
(strategy) => strategy.disabled,
); );
if (hasOnlyDisabledStrategies) {
// Disable the feature in the environment if it only has disabled strategies
await this.unprotectedUpdateEnabled(
projectId,
featureName,
environment,
false,
auditUser,
);
}
const preData = this.featureStrategyToPublic(existingStrategy); const preData = this.featureStrategyToPublic(existingStrategy);
await this.eventService.storeEvent( await this.eventService.storeEvent(