Skip to content

Commit 4fcd0eb

Browse files
committed
Don't store summary stats fetched in the Analytics Hub
1 parent 2fdda5a commit 4fcd0eb

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,8 @@ private extension AnalyticsHubViewModel {
215215
let action = StatsActionV4.retrieveSiteSummaryStats(siteID: siteID,
216216
period: period,
217217
quantity: timeRangeSelectionType.quantity,
218-
latestDateToInclude: latestDateToInclude) { result in
218+
latestDateToInclude: latestDateToInclude,
219+
saveInStorage: false) { result in
219220
continuation.resume(with: result)
220221
}
221222
stores.dispatch(action)

WooCommerce/WooCommerceTests/ViewRelated/Dashboard/Analytics Hub/AnalyticsHubViewModelTests.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ final class AnalyticsHubViewModelTests: XCTestCase {
2626
case let .retrieveTopEarnerStats(_, _, _, _, _, _, _, completion):
2727
let topEarners = TopEarnerStats.fake().copy(items: [.fake()])
2828
completion(.success(topEarners))
29-
case let .retrieveSiteSummaryStats(_, _, _, _, completion):
29+
case let .retrieveSiteSummaryStats(_, _, _, _, _, completion):
3030
let siteStats = SiteSummaryStats.fake().copy(visitors: 30, views: 53)
3131
completion(.success(siteStats))
3232
default:
@@ -61,7 +61,7 @@ final class AnalyticsHubViewModelTests: XCTestCase {
6161
completion(.failure(NSError(domain: "Test", code: 1)))
6262
case let .retrieveTopEarnerStats(_, _, _, _, _, _, _, completion):
6363
completion(.failure(NSError(domain: "Test", code: 1)))
64-
case let .retrieveSiteSummaryStats(_, _, _, _, completion):
64+
case let .retrieveSiteSummaryStats(_, _, _, _, _, completion):
6565
completion(.failure(NSError(domain: "Test", code: 1)))
6666
default:
6767
break
@@ -99,7 +99,7 @@ final class AnalyticsHubViewModelTests: XCTestCase {
9999
case let .retrieveTopEarnerStats(_, _, _, _, _, _, _, completion):
100100
let topEarners = TopEarnerStats.fake().copy(items: [.fake()])
101101
completion(.success(topEarners))
102-
case let .retrieveSiteSummaryStats(_, _, _, _, completion):
102+
case let .retrieveSiteSummaryStats(_, _, _, _, _, completion):
103103
let siteStats = SiteSummaryStats.fake()
104104
loadingSessionsCard = vm.sessionsCard
105105
completion(.success(siteStats))

0 commit comments

Comments
 (0)