Skip to content

Commit 58ba451

Browse files
committed
Track barcode scan error with a full error object
1 parent 5feddab commit 58ba451

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

WooCommerce/Classes/POS/Analytics/WooAnalyticsEvent+PointOfSale.swift

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ extension WooAnalyticsEvent {
2525
static let resultsCount = "results_count"
2626
static let millisecondsSinceRequestSent = "milliseconds_since_request_sent"
2727
static let totalItems = "total_items"
28-
static let error = "error"
2928
}
3029

3130
static func paymentsOnboardingShown() -> WooAnalyticsEvent {
@@ -42,7 +41,7 @@ extension WooAnalyticsEvent {
4241
sourceViewType: WooAnalyticsEvent.PointOfSale.SourceViewType,
4342
itemType: WooAnalyticsEvent.PointOfSale.ItemType,
4443
productType: WooAnalyticsEvent.PointOfSale.CartItemProductType? = nil,
45-
error: String? = nil
44+
error: Error? = nil
4645
) -> WooAnalyticsEvent {
4746
var properties: [String: String] = [
4847
Key.sourceViewType: sourceViewType.rawValue,
@@ -57,13 +56,10 @@ extension WooAnalyticsEvent {
5756
properties[Key.productType] = productType.rawValue
5857
}
5958

60-
if let error {
61-
properties[Key.error] = error
62-
}
63-
6459
return WooAnalyticsEvent(
6560
statName: .pointOfSaleAddItemToCart,
66-
properties: properties
61+
properties: properties,
62+
error: error
6763
)
6864
}
6965

WooCommerce/Classes/POS/Models/PointOfSaleAggregateModel.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ extension PointOfSaleAggregateModel {
217217
event: .PointOfSale.addItemToCart(
218218
sourceViewType: .scanner,
219219
itemType: .error,
220-
error: error.localizedDescription
220+
error: error
221221
)
222222
)
223223
}

0 commit comments

Comments
 (0)