remove pointOfSaleSettingsi1

Author: iamgabrielma

Modules/Sources/Experiments/DefaultFeatureFlagService.swift

@@ -90,8 +90,6 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
             return true
         case .pointOfSaleOrdersi2:
             return true
-        case .pointOfSaleSettingsi1:
-            return true
         case .orderAddressMapSearch:
             return true
         case .pointOfSaleHistoricalOrdersi1:

Modules/Sources/Experiments/FeatureFlag.swift

@@ -187,10 +187,6 @@ public enum FeatureFlag: Int {
     ///
     case pointOfSaleOrdersi2
 
-    /// Enables the entry point for Point of Sale Settings
-    ///
-    case pointOfSaleSettingsi1
-
     /// Enables the CTA to search for an address in the map in order details > shipping address.
     ///
     case orderAddressMapSearch

WooCommerce/Classes/POS/Presentation/CartView.swift

@@ -23,10 +23,6 @@ struct CartView: View {
         abs(offSetPosition) < maxOffset
     }
 
-    private var isPOSSettingsEnabled: Bool {
-        featureFlags.isFeatureFlagEnabled(.pointOfSaleSettingsi1)
-    }
-
     @State private var showBarcodeScanningModal: Bool = false
 
     var body: some View {
@@ -142,10 +138,6 @@ private extension CartView {
             "pos.cartView.cartTitle",
             value: "Cart",
             comment: "Title at the header for the Cart view.")
-        static let addItemsToCartHint = NSLocalizedString(
-            "pos.cartView.addItemsToCartHint",
-            value: "Tap on a product to \n add it to the cart",
-            comment: "Hint to add products to the Cart when this is empty.")
         static let addItemsToCartOrScanHint = NSLocalizedString(
             "pos.cartView.addItemsToCartOrScanHint",
             value: "Tap on a product to \n add it to the cart, or ",
@@ -206,7 +198,7 @@ private extension CartView {
             // SwiftUI doesn't allow us to absolutely pin a view to the centre then position other views relative to it
             // Instead, we can centre the text, and then put the image in an offset overlay. Offsetting from the top
             // avoids issues when the text size is changed through dynamic type.
-            Text(isPOSSettingsEnabled ? Localization.addItemsToCartOrScanHint : Localization.addItemsToCartHint)
+            Text(Localization.addItemsToCartOrScanHint)
                 .font(Constants.secondaryFont)
                 .foregroundColor(Color.posOnSurfaceVariantLowest)
                 .multilineTextAlignment(.center)
@@ -217,7 +209,6 @@ private extension CartView {
                         .offset(y: -(Constants.shoppingBagImageSize + Constants.emptyViewImageTextSpacing))
                         .aspectRatio(contentMode: .fit)
                 }
-            if isPOSSettingsEnabled {
                 Button(action: {
                     analytics.track(.pointOfSaleEmptyCartSetupScannerTapped)
                     showBarcodeScanningModal = true
@@ -229,7 +220,6 @@ private extension CartView {
                         Image(systemName: "barcode.viewfinder")
                     }
                 })
-            }
             Spacer()
         }
         .background(backgroundColor.ignoresSafeArea(.all))

WooCommerce/Classes/POS/Presentation/POSFloatingControlView.swift

@@ -24,18 +24,10 @@ struct POSFloatingControlView: View {
         self._showSettings = showSettings
     }
 
-    private var isPOSSettingsEnabled: Bool {
-        featureFlags.isFeatureFlagEnabled(.pointOfSaleSettingsi1)
-    }
-
     var body: some View {
         HStack {
             Menu {
-                if isPOSSettingsEnabled {
-                    compactOptions()
-                } else {
-                    completeOptions()
-                }
+                compactOptions()
             } label: {
                 VStack {
                     Spacer()
@@ -85,87 +77,15 @@ private extension POSFloatingControlView {
                 icon: { Image(systemName: "rectangle.portrait.and.arrow.forward") }
             )
         }
-        if featureFlags.isFeatureFlagEnabled(.pointOfSaleSettingsi1) {
-            Button {
-                analytics.track(.pointOfSaleSettingsMenuItemTapped)
-                showSettings = true
-            } label: {
-                Label(
-                    title: { Text(Localization.settings) },
-                    icon: { Image(systemName: "gearshape") }
-                )
-            }
-        }
-
-        if featureFlags.isFeatureFlagEnabled(.pointOfSaleHistoricalOrdersi1) {
-            Button {
-                analytics.track(event: WooAnalyticsEvent.PointOfSale.ordersMenuItemTapped())
-                showOrders = true
-            } label: {
-                Label(
-                    title: { Text(Localization.orders) },
-                    icon: { Image(systemName: "text.document") }
-                )
-            }
-        }
-    }
-
-    @ViewBuilder private func completeOptions() -> some View {
         Button {
-            analytics.track(.pointOfSaleExitMenuItemTapped)
-            showExitPOSModal = true
+            analytics.track(.pointOfSaleSettingsMenuItemTapped)
+            showSettings = true
         } label: {
             Label(
-                title: { Text(Localization.exitPointOfSale) },
-                icon: { Image(systemName: "rectangle.portrait.and.arrow.forward") }
+                title: { Text(Localization.settings) },
+                icon: { Image(systemName: "gearshape") }
             )
         }
-        Button {
-            analytics.track(.pointOfSaleGetSupportTapped)
-            showSupport = true
-        } label: {
-            Label(
-                title: { Text(Localization.getSupport) },
-                icon: { Image(systemName: "questionmark.circle") }
-            )
-        }
-        Button {
-            showDocumentation = true
-            analytics.track(.pointOfSaleViewDocsTapped)
-        } label: {
-            Label(
-                title: { Text(Localization.viewDocumentation) },
-                icon: { Image(systemName: "info.circle") }
-            )
-        }
-        Button {
-            showProductRestrictionsModal = true
-            analytics.track(.pointOfSaleSimpleProductsExplanationDialogShown)
-        } label: {
-            Label(
-                title: { Text(Localization.productRestrictionsInfo) },
-                icon: { Image(systemName: "magnifyingglass") })
-        }
-        Button {
-            showBarcodeScanningModal = true
-            analytics.track(.pointOfSaleBarcodeScanningMenuItemTapped)
-        } label: {
-            Label(
-                title: {
-                    Text(Localization.barcodeScanningSetup)
-                },
-                icon: { Image(systemName: "barcode.viewfinder") })
-        }
-        if featureFlags.isFeatureFlagEnabled(.pointOfSaleSettingsi1) {
-            Button {
-                showSettings = true
-            } label: {
-                Label(
-                    title: { Text(Localization.settings) },
-                    icon: { Image(systemName: "gearshape") }
-                )
-            }
-        }
 
         if featureFlags.isFeatureFlagEnabled(.pointOfSaleHistoricalOrdersi1) {
             Button {