Merge branch 'trunk' into issue/7904-simplified-login-ab-testing

Author: itsmeichigo

CodeGeneration/Sourcery/Copiable/Models+Copiable.swifttemplate

@@ -183,7 +183,7 @@ let specsToGenerate: [CopiableSpec] = matchingTypes.map { type in
     }
 
     return CopiableSpec(
-        name: type.name,
+        name: type.globalName,
         accessLevelWithSpacePostfix: type.accessLevel == "internal" ? "" : "\(type.accessLevel) ",
         properties: propSpecs
     )

CodeGeneration/Sourcery/Fakes/Fakes.swifttemplate

@@ -69,7 +69,7 @@ let specsToGenerate: [FakeableSpec] = matchingTypes.map { type in
         )
     }
 
-    return FakeableSpec(name: type.name, accessLevelWithSpacePostfix: accessLevel, content: .initializer(propSpecs))
+    return FakeableSpec(name: type.globalName, accessLevelWithSpacePostfix: accessLevel, content: .initializer(propSpecs))
 }
 -%>
 <%#

Experiments/Experiments/ABTest.swift

@@ -20,6 +20,10 @@ public enum ABTest: String, CaseIterable {
     ///
     case abTestLoginWithWPComOnly = "woocommerceios_login_wpcom_only"
 
+    /// A/B test for the Products Onboarding banner on the My Store dashboard.
+    /// Experiment ref: pbxNRc-26F-p2
+    case productsOnboardingBanner = "woocommerceios_products_onboarding_first_product_banner"
+
     /// Returns a variation for the given experiment
     public var variation: Variation {
         ExPlat.shared?.experiment(rawValue) ?? .control
@@ -30,7 +34,7 @@ public enum ABTest: String, CaseIterable {
     /// When adding a new experiment, add it to the appropriate case depending on its context (logged-in or logged-out experience).
     public var context: ExperimentContext {
         switch self {
-        case .aaTestLoggedIn202210:
+        case .aaTestLoggedIn202210, .productsOnboardingBanner:
             return .loggedIn
         case .aaTestLoggedOut, .abTestLoginWithWPComOnly:
             return .loggedOut

Experiments/Experiments/DefaultFeatureFlagService.swift

@@ -18,7 +18,7 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
         case .shippingLabelsOnboardingM1:
             return buildConfig == .localDeveloper || buildConfig == .alpha
         case .newToWooCommerceLinkInLoginPrologue:
-            return true
+            return false
         case .loginPrologueOnboarding:
             return true
         case .loginErrorNotifications:
@@ -33,8 +33,6 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
             return true
         case .searchProductsBySKU:
             return true
-        case .orderCreationSearchCustomers:
-            return buildConfig == .localDeveloper || buildConfig == .alpha
         case .wpcomSignup:
             return false
         case .inAppPurchases:
@@ -45,6 +43,14 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
             return buildConfig == .localDeveloper || buildConfig == .alpha
         case .productsOnboarding:
             return buildConfig == .localDeveloper || buildConfig == .alpha
+        case .performanceMonitoring,
+                .performanceMonitoringCoreData,
+                .performanceMonitoringFileIO,
+                .performanceMonitoringNetworking,
+                .performanceMonitoringViewController,
+                .performanceMonitoringUserInteraction:
+            // Disabled by default to avoid costs spikes, unless in internal testing builds.
+            return buildConfig == .alpha
         default:
             return true
         }

Experiments/Experiments/FeatureFlag.swift

@@ -70,10 +70,6 @@ public enum FeatureFlag: Int {
     ///
     case searchProductsBySKU
 
-    /// Enables the Search Customers functionality in the Order Creation screen
-    ///
-    case orderCreationSearchCustomers
-
     /// Enables signing up for a WP.com account.
     ///
     case wpcomSignup
@@ -93,4 +89,39 @@ public enum FeatureFlag: Int {
     /// Hides products onboarding development.
     ///
     case productsOnboarding
+
+    // MARK: - Performance Monitoring
+    //
+    // These flags are not transient. That is, they are not here to help us rollout a feature,
+    // but to serve a safety switches to granularly turn off performance monitoring if it looks
+    // like we are consuming too many events.
+
+    /// Whether to enable performance monitoring.
+    ///
+    case performanceMonitoring
+
+    /// Whether to enable performance monitoring for Core Data operations.
+    ///
+    /// - Note: The app will ignore this if `performanceMonitoring` is `false`
+    case performanceMonitoringCoreData
+
+    /// Whether to enable performance monitoring for file IO operations.
+    ///
+    /// - Note: The app will ignore this if `performanceMonitoring` is `false`
+    case performanceMonitoringFileIO
+
+    /// Whether to enable performance monitoring for networking operations.
+    ///
+    /// - Note: The app will ignore this if `performanceMonitoring` is `false`
+    case performanceMonitoringNetworking
+
+    /// Whether to enable performance monitoring for user interaction events.
+    ///
+    /// - Note: The app will ignore this if `performanceMonitoring` is `false`
+    case performanceMonitoringUserInteraction
+
+    /// Whether to enable performance monitoring for `UIViewController` life-cycle events.
+    ///
+    /// - Note: The app will ignore this if `performanceMonitoring` is `false`.
+    case performanceMonitoringViewController
 }

Fakes/Fakes/Hardware.generated.swift

@@ -12,10 +12,10 @@ extension CardBrand {
         .visa
     }
 }
-extension CardPresentReceiptParameters {
+extension Hardware.CardPresentReceiptParameters {
     /// Returns a "ready to use" type filled with fake values.
     ///
-    public static func fake() -> CardPresentReceiptParameters {
+    public static func fake() -> Hardware.CardPresentReceiptParameters {
         .init(
             amount: .fake(),
             formattedAmount: .fake(),
@@ -27,10 +27,10 @@ extension CardPresentReceiptParameters {
         )
     }
 }
-extension CardPresentTransactionDetails {
+extension Hardware.CardPresentTransactionDetails {
     /// Returns a "ready to use" type filled with fake values.
     ///
-    public static func fake() -> CardPresentTransactionDetails {
+    public static func fake() -> Hardware.CardPresentTransactionDetails {
         .init(
             last4: .fake(),
             expMonth: .fake(),
@@ -44,10 +44,10 @@ extension CardPresentTransactionDetails {
         )
     }
 }
-extension Charge {
+extension Hardware.Charge {
     /// Returns a "ready to use" type filled with fake values.
     ///
-    public static func fake() -> Charge {
+    public static func fake() -> Hardware.Charge {
         .init(
             id: .fake(),
             amount: .fake(),
@@ -66,10 +66,10 @@ extension ChargeStatus {
         .succeeded
     }
 }
-extension PaymentIntent {
+extension Hardware.PaymentIntent {
     /// Returns a "ready to use" type filled with fake values.
     ///
-    public static func fake() -> PaymentIntent {
+    public static func fake() -> Hardware.PaymentIntent {
         .init(
             id: .fake(),
             status: .fake(),