Skip to content

Commit dd25daf

Browse files
authored
Merge pull request #5297 from woocommerce/ec/order-remove-flag
2 parents 5a7814f + 5366af2 commit dd25daf

File tree

3 files changed

+4
-10
lines changed

3 files changed

+4
-10
lines changed

WooCommerce/Classes/ViewRelated/Dashboard/Settings/Beta features/BetaFeaturesViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ private extension BetaFeaturesViewController {
9090
/// A section is returned only when the store is ready to receive payments
9191
///
9292
func ordersSection() -> Section? {
93-
guard ServiceLocator.featureFlagService.isFeatureFlagEnabled(.quickOrderPrototype), paymentsStoreUseCase.state == .completed else {
93+
guard paymentsStoreUseCase.state == .completed else {
9494
return nil
9595
}
9696

WooCommerce/Classes/ViewRelated/Orders/OrderListViewModel.swift

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,6 @@ final class OrderListViewModel {
9090
///
9191
@Published var hideQuickOrderBanners: Bool = false
9292

93-
/// Tracks if the Quick Order feature is ready to be released to the public
94-
///
95-
private let isQuickOrderDevelopmentComplete = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.quickOrderPrototype)
96-
9793
init(siteID: Int64,
9894
stores: StoresManager = ServiceLocator.stores,
9995
storageManager: StorageManagerType = ServiceLocator.storageManager,
@@ -251,14 +247,13 @@ extension OrderListViewModel {
251247
let errorState = $hasErrorLoadingData.removeDuplicates()
252248
let experimentalState = $isQuickOrderEnabled.removeDuplicates()
253249
Publishers.CombineLatest4(enrolledState, errorState, experimentalState, $hideQuickOrderBanners)
254-
.map { [weak self] enrolledState, hasError, isQuickOrderEnabled, hasDismissedBanners -> TopBanner in
255-
guard let self = self else { return .none }
250+
.map { enrolledState, hasError, isQuickOrderEnabled, hasDismissedBanners -> TopBanner in
256251

257252
guard !hasError else {
258253
return .error
259254
}
260255

261-
guard self.isQuickOrderDevelopmentComplete, !hasDismissedBanners else {
256+
guard !hasDismissedBanners else {
262257
return .none
263258
}
264259

WooCommerce/Classes/ViewRelated/Orders/OrdersRootViewController.swift

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,9 +110,8 @@ private extension OrdersRootViewController {
110110
///
111111
func configureNavigationButtons(isQuickOrderExperimentalToggleEnabled: Bool) {
112112
let shouldShowQuickOrderButton: Bool = {
113-
let isQuickOrderEnabled = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.quickOrderPrototype) && isQuickOrderExperimentalToggleEnabled
114113
let isInPersonPaymentsConfigured = inPersonPaymentsUseCase.state == .completed
115-
return isQuickOrderEnabled && isInPersonPaymentsConfigured
114+
return isQuickOrderExperimentalToggleEnabled && isInPersonPaymentsConfigured
116115
}()
117116
let buttons: [UIBarButtonItem?] = [
118117
ordersViewController.createSearchBarButtonItem(),

0 commit comments

Comments
 (0)