Skip to content

Commit 26e5d20

Browse files
authored
Merge pull request #9075 from woocommerce/order-creation-analytics-update
Add new prop to `ORDER_CREATION_SUCCESS` event
2 parents 0b93fea + 7f7d31a commit 26e5d20

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

WooCommerce/src/main/kotlin/com/woocommerce/android/analytics/AnalyticsTracker.kt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,8 @@ class AnalyticsTracker private constructor(private val context: Context) {
259259
const val KEY_TIME_ELAPSED_SINCE_ADD_NEW_ORDER_IN_MILLIS = "milliseconds_since_order_add_new"
260260
const val KEY_TIME_ELAPSED_SINCE_CARD_COLLECT_PAYMENT_IN_MILLIS = "milliseconds_since_card_collect_payment_flow"
261261

262+
const val KEY_COUPONS_COUNT = "coupons_count"
263+
262264
enum class OrderNoteType(val value: String) {
263265
CUSTOMER("customer"),
264266
PRIVATE("private"),

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/orders/creation/OrderCreateEditViewModel.kt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import com.woocommerce.android.analytics.AnalyticsEvent.ORDER_STATUS_CHANGE
3232
import com.woocommerce.android.analytics.AnalyticsEvent.PRODUCT_SEARCH_VIA_SKU_FAILURE
3333
import com.woocommerce.android.analytics.AnalyticsEvent.PRODUCT_SEARCH_VIA_SKU_SUCCESS
3434
import com.woocommerce.android.analytics.AnalyticsTracker
35+
import com.woocommerce.android.analytics.AnalyticsTracker.Companion.KEY_COUPONS_COUNT
3536
import com.woocommerce.android.analytics.AnalyticsTracker.Companion.KEY_ERROR_CONTEXT
3637
import com.woocommerce.android.analytics.AnalyticsTracker.Companion.KEY_ERROR_DESC
3738
import com.woocommerce.android.analytics.AnalyticsTracker.Companion.KEY_ERROR_TYPE
@@ -630,11 +631,12 @@ class OrderCreateEditViewModel @Inject constructor(
630631
private fun trackOrderCreationSuccess() {
631632
tracker.track(
632633
ORDER_CREATION_SUCCESS,
633-
mutableMapOf<String, String>().also { mutableMap ->
634+
mutableMapOf<String, Any>().also { mutableMap ->
634635
OrderDurationRecorder.millisecondsSinceOrderAddNew().getOrNull()?.let { timeElapsed ->
635636
mutableMap[AnalyticsTracker.KEY_TIME_ELAPSED_SINCE_ADD_NEW_ORDER_IN_MILLIS] =
636637
timeElapsed.toString()
637638
}
639+
mutableMap[KEY_COUPONS_COUNT] = orderDraft.value?.couponLines?.size ?: 0
638640
}
639641
)
640642
}

0 commit comments

Comments
 (0)