Skip to content

Commit fb5b514

Browse files
authored
Merge pull request #6640 from woocommerce/fix/1183-remove-stats-api-workaround
Remove stats API workaround for stale data
2 parents 1f1aa7f + 8b1597e commit fb5b514

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

Networking/Networking/Remote/OrderStatsRemoteV4.swift

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,10 @@ public final class OrderStatsRemoteV4: Remote {
2121
completion: @escaping (Result<OrderStatsV4, Error>) -> Void) {
2222
let dateFormatter = DateFormatter.Defaults.iso8601WithoutTimeZone
2323

24-
// Workaround for #1183: random number between 31-100 for `num_page` param.
25-
// Replace `randomQuantity` with `quantity` in `num_page` param when API issue is fixed.
26-
let randomQuantity = arc4random_uniform(70) + 31
27-
2824
let parameters = [ParameterKeys.interval: unit.rawValue,
2925
ParameterKeys.after: dateFormatter.string(from: earliestDateToInclude),
3026
ParameterKeys.before: dateFormatter.string(from: latestDateToInclude),
31-
ParameterKeys.quantity: String(randomQuantity)]
27+
ParameterKeys.quantity: String(quantity)]
3228

3329
let request = JetpackRequest(wooApiVersion: .wcAnalytics, method: .get, siteID: siteID, path: Constants.orderStatsPath, parameters: parameters)
3430
let mapper = OrderStatsV4Mapper(siteID: siteID, granularity: unit)

0 commit comments

Comments
 (0)