Skip to content

Commit 3571be9

Browse files
committed
move setHasPOSBeenOpenedAtLeastOnce to the correct place
1 parent d43685b commit 3571be9

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

WooCommerce/Classes/POS/TabBar/POSTabCoordinator.swift

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,11 +116,19 @@ final class POSTabCoordinator {
116116
}
117117

118118
func onTabSelected() {
119+
setPOSHasBeenOpened()
119120
presentPOSView(siteID: siteID)
120121
}
121122
}
122123

123124
private extension POSTabCoordinator {
125+
func setPOSHasBeenOpened() {
126+
Task { @MainActor in
127+
let action = AppSettingsAction.setHasPOSBeenOpenedAtLeastOnce { _ in }
128+
storesManager.dispatch(action)
129+
}
130+
}
131+
124132
func presentPOSView(siteID: Int64) {
125133
Task { @MainActor [weak self] in
126134
guard let self else { return }

WooCommerce/Classes/ViewRelated/AppCoordinator.swift

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,9 +107,6 @@ private extension AppCoordinator {
107107
func schedulePOSSurveyNotificationIfNeeded() {
108108
Task { @MainActor in
109109
await POSNotificationScheduler(stores: stores).scheduleLocalNotificationIfEligible(for: .currentMerchant)
110-
111-
let action = AppSettingsAction.setHasPOSBeenOpenedAtLeastOnce { _ in }
112-
stores.dispatch(action)
113110
}
114111
}
115112
}

0 commit comments

Comments
 (0)