Skip to content

Commit 026b088

Browse files
authored
Merge pull request #8306 from woocommerce/issue/fix-background-warnings
Analytics Hub: Fix UI access from a background thread.
2 parents ad715e5 + ca9c6ad commit 026b088

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

WooCommerce/Classes/ViewRelated/Dashboard/Analytics Hub/AnalyticsHubViewModel.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ final class AnalyticsHubViewModel: ObservableObject {
7171
do {
7272
try await retrieveOrderStats()
7373
} catch {
74-
switchToErrorState()
74+
await switchToErrorState()
7575
DDLogWarn("⚠️ Error fetching analytics data: \(error)")
7676
}
7777
}
@@ -148,12 +148,14 @@ private extension AnalyticsHubViewModel {
148148
// MARK: Data - UI mapping
149149
private extension AnalyticsHubViewModel {
150150

151+
@MainActor
151152
func switchToLoadingState() {
152153
self.revenueCard = revenueCard.redacted
153154
self.ordersCard = ordersCard.redacted
154155
self.productCard = productCard.redacted
155156
}
156157

158+
@MainActor
157159
func switchToErrorState() {
158160
self.currentOrderStats = nil
159161
self.previousOrderStats = nil

0 commit comments

Comments
 (0)