From 85cad5a6ce041657a6340ac4240aff58ca076275 Mon Sep 17 00:00:00 2001 From: CarlNation Date: Tue, 7 Oct 2025 14:57:12 -0400 Subject: [PATCH 1/3] CASH-1637 CASH-1637 correcting fields iteratively since dev environment is unstable --- src/mixins/analytics-mixin.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/mixins/analytics-mixin.js b/src/mixins/analytics-mixin.js index 74b74283b..50115bbf9 100644 --- a/src/mixins/analytics-mixin.js +++ b/src/mixins/analytics-mixin.js @@ -221,7 +221,7 @@ export default { ? order?.serviceLocation?.provider?.address?.zipCodeCtu : order?.serviceLocation?.zipCodeCtu; label.subTotalPrice = getSubTotal(order?.lineItems); - label.totalPrice = getAmountDue(order?.lineItems); + label.totalPrice = getAmountDue(order?.lineItems, true); await this.pushEventToGA( GaCategories.FMG_SESSION_DATA, @@ -237,9 +237,14 @@ export default { (lineItem) => lineItem.partType == partTypeStrings.EARLY_BIRD ); + const promoCodes = order?.lineItems?.promos + ?.map((item) => item.promoCode) + .filter((code) => code) + .join(", "); + label = {}; label.serviceType = order?.serviceLocation?.appointmentType; - label.promoCodes = order?.lineItems?.promos; + label.promoCodes = promoCodes; label.hasTechnicianNotes = order?.serviceLocation?.techNotes ? true : false; label.paymentMethod = order?.payment?.piaType; label.isTextingOptedIn = order?.customer?.isSmsOptIn ? true : false; From 58e2ec0addacd163c7467c8516e6b9e8abec8e44 Mon Sep 17 00:00:00 2001 From: CarlNation Date: Tue, 7 Oct 2025 15:06:41 -0400 Subject: [PATCH 2/3] CASH-1637 CASH-1637 check for array to prevent exception when not --- src/mixins/analytics-mixin.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/mixins/analytics-mixin.js b/src/mixins/analytics-mixin.js index 50115bbf9..3ff326e1c 100644 --- a/src/mixins/analytics-mixin.js +++ b/src/mixins/analytics-mixin.js @@ -237,10 +237,12 @@ export default { (lineItem) => lineItem.partType == partTypeStrings.EARLY_BIRD ); - const promoCodes = order?.lineItems?.promos - ?.map((item) => item.promoCode) - .filter((code) => code) - .join(", "); + const promoCodes = Array.isArray(order?.lineItems?.promos) + ? order?.lineItems?.promos + ?.map((item) => item.promoCode) + .filter((code) => code) + .join(", ") + : ""; label = {}; label.serviceType = order?.serviceLocation?.appointmentType; From 7f28449d1f479c98a078ecceaed2a196ea40d9c8 Mon Sep 17 00:00:00 2001 From: CarlNation Date: Tue, 7 Oct 2025 15:10:18 -0400 Subject: [PATCH 3/3] CASH-1637 CASH-1637 optional chaining ? not necessary since checking for an array already --- src/mixins/analytics-mixin.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mixins/analytics-mixin.js b/src/mixins/analytics-mixin.js index 3ff326e1c..f5a7d10e0 100644 --- a/src/mixins/analytics-mixin.js +++ b/src/mixins/analytics-mixin.js @@ -239,7 +239,7 @@ export default { const promoCodes = Array.isArray(order?.lineItems?.promos) ? order?.lineItems?.promos - ?.map((item) => item.promoCode) + .map((item) => item.promoCode) .filter((code) => code) .join(", ") : "";