Merge pull request #5 from managed-components/ZRZ-1281

add nullchecks before using toString
managed-components · Apr 26, 2024 · 3744442 · 3744442
2 parents be8fe03 + 646d2c6
commit 3744442
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -51,7 +51,7 @@ const getEcommercePayload = (event: MCEvent) => {
  .join()
  payload.contents = payload.products.map((product: Product) => ({
  id: product.product_id,
- item_price: product.price.toString(),
+ item_price: product.price?.toString() || '',
  quantity: product.quantity,
  }))
  payload.num_items =
@@ -98,7 +98,7 @@ export const getEventData = async (
  language: payload.language || client.language.split(',')[0].substring(0, 2),
  ...remainingEventData,
  user_data: {
- client_ip_address: payload.client_ip_address || client.ip.toString(),
+ client_ip_address: payload.client_ip_address || client.ip?.toString(),
  client_user_agent: payload.client_user_agent || parsedUserAgent.ua,
  ...(payload.click_id && { click_id: payload.click_id }),
  ...hashedUserProperties,