Merge pull request #8495 from woocommerce/issue/refactor-font-helpers

Ecarrion · web-flow · commit 148b0a955a83 · 2022-12-28T11:36:44.000-05:00
Variations: Adds feature flag
diff --git a/Experiments/Experiments/DefaultFeatureFlagService.swift b/Experiments/Experiments/DefaultFeatureFlagService.swift
@@ -58,6 +58,8 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
         case .applicationPasswordAuthenticationForSiteCredentialLogin:
             // Enable this to test application password authentication (WIP)
             return false
+        case .generateAllVariations:
+            return buildConfig == .localDeveloper || buildConfig == .alpha
         default:
             return true
         }
diff --git a/Experiments/Experiments/FeatureFlag.swift b/Experiments/Experiments/FeatureFlag.swift
@@ -137,4 +137,8 @@ public enum FeatureFlag: Int {
     /// Whether application password authentication should be used when a user logs in with site credentials.
     ///
     case applicationPasswordAuthenticationForSiteCredentialLogin
+
+    /// Allows merchants to create all variations from a single button
+    ///
+    case generateAllVariations
 }

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,8 @@ public struct DefaultFeatureFlagService: FeatureFlagService {`
`58`	`58`	`case .applicationPasswordAuthenticationForSiteCredentialLogin:`
`59`	`59`	`// Enable this to test application password authentication (WIP)`
`60`	`60`	`return false`
	`61`	`+ case .generateAllVariations:`
	`62`	`+ return buildConfig == .localDeveloper \|\| buildConfig == .alpha`
`61`	`63`	`default:`
`62`	`64`	`return true`
`63`	`65`	`}`
Original file line number	Diff line number	Diff line change
`@@ -137,4 +137,8 @@ public enum FeatureFlag: Int {`
`137`	`137`	`/// Whether application password authentication should be used when a user logs in with site credentials.`
`138`	`138`	`///`
`139`	`139`	`case applicationPasswordAuthenticationForSiteCredentialLogin`
	`140`	`+`
	`141`	`+ /// Allows merchants to create all variations from a single button`
	`142`	`+ ///`
	`143`	`+ case generateAllVariations`
`140`	`144`	`}`