From af3190a8f24f01d620cc5f92495349a61e657237 Mon Sep 17 00:00:00 2001
From: Erik <9088720+erikdstock@users.noreply.github.com>
Date: Tue, 11 Feb 2025 18:25:19 -0600
Subject: [PATCH] update estimate widget tracking (#15194)

---
 package.json                             |  2 +-
 src/Components/ArtsyShippingEstimate.tsx | 52 +++++++++++++++---------
 yarn.lock                                |  2 +-
 3 files changed, 34 insertions(+), 22 deletions(-)

diff --git a/package.json b/package.json
index cd204f2dd5f..1eaaa3294cb 100644
--- a/package.json
+++ b/package.json
@@ -69,7 +69,7 @@
     "styled-components": "^6.1.13"
   },
   "dependencies": {
-    "@artsy/cohesion": "4.233.0",
+    "@artsy/cohesion": "^4.233.0",
     "@artsy/commerce_helpers": "artsy/commerce_helpers",
     "@artsy/detect-responsive-traits": "^0.1.0",
     "@artsy/dismissible": "^0.3.2",
diff --git a/src/Components/ArtsyShippingEstimate.tsx b/src/Components/ArtsyShippingEstimate.tsx
index da0772eb2b5..42ef367ec1d 100644
--- a/src/Components/ArtsyShippingEstimate.tsx
+++ b/src/Components/ArtsyShippingEstimate.tsx
@@ -81,6 +81,7 @@ export const ArtsyShippingEstimate = ({
       context_page_owner_id: contextPageOwnerId,
       context_page_owner_slug: contextPageOwnerSlug,
       origin: artworkData.shippingOrigin,
+      estimate_available: estimate.estimateAvailable,
       destination: estimate.destination,
       minimum_estimate: estimate.minPrice,
       maximum_estimate: estimate.maxPrice,
@@ -185,10 +186,11 @@ export const ArtsyShippingEstimate = ({
 }
 
 interface PriceEstimate {
-  destination: string
-  minPrice: number
+  destination: string | null
+  estimateAvailable: boolean
+  minPrice: number | null
   maxPrice?: number | null
-  currency: string
+  currency: string | null
 }
 
 interface UseWidgetObserverProps {
@@ -202,6 +204,17 @@ const useWidgetObserver = ({
   const [visiblePrice, setVisiblePrice] = useState<PriceEstimate | null>(null)
 
   const extractEstimateFromDom = useCallback((): PriceEstimate | null => {
+    const destinationEl = document.getElementsByClassName(
+      "artajs__modal__quotes__destination",
+    )[0]
+    // e.g. "Chicago, IL, US (destination)"
+    const destination =
+      destinationEl?.textContent?.match(/(.+)\(destination\)/)?.[1]
+
+    if (!destination) {
+      return null
+    }
+
     const priceAmountEl = document.getElementsByClassName(
       "artajs__modal__quotes__price__amount",
     )[0]
@@ -214,18 +227,13 @@ const useWidgetObserver = ({
         .map(n => Number.parseFloat(n)) || []
 
     if (!minPrice) {
-      return null
-    }
-
-    const destinationEl = document.getElementsByClassName(
-      "artajs__modal__quotes__destination",
-    )[0]
-    // e.g. "Chicago, IL, US (destination)"
-    const destination =
-      destinationEl?.textContent?.match(/(.+)\(destination\)/)?.[1]
-
-    if (!destination) {
-      return null
+      return {
+        estimateAvailable: false,
+        destination,
+        minPrice: null,
+        maxPrice: null,
+        currency: null,
+      }
     }
 
     const priceCurrencyEl = document.getElementsByClassName(
@@ -233,16 +241,18 @@ const useWidgetObserver = ({
     )[0]
     const currency = priceCurrencyEl?.textContent
 
-    if (!currency) {
-      return null
+    return {
+      minPrice,
+      maxPrice,
+      currency,
+      destination,
+      estimateAvailable: true,
     }
-
-    return { minPrice, maxPrice, currency, destination }
   }, [])
 
   // biome-ignore lint/correctness/useExhaustiveDependencies: We only want to run this when the price changes
   useEffect(() => {
-    if (visiblePrice?.minPrice && visiblePrice?.maxPrice) {
+    if (visiblePrice) {
       onViewEstimatedPrice(visiblePrice)
     }
   }, [visiblePrice])
@@ -259,6 +269,7 @@ const useWidgetObserver = ({
     // Callback function to execute when mutations are observed
     const callback = (_mutationList, _observer) => {
       const price = extractEstimateFromDom()
+
       if (price) {
         setVisiblePrice(price)
       }
@@ -274,6 +285,7 @@ const useWidgetObserver = ({
 
   const disconnectWidgetObserver = useCallback(() => {
     widgetObserver.current?.disconnect()
+    setVisiblePrice(null)
   }, [])
 
   return {
diff --git a/yarn.lock b/yarn.lock
index e979eed6e1c..5167e9cd42c 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -22,7 +22,7 @@
   dependencies:
     preact "^10.11.3"
 
-"@artsy/cohesion@4.233.0":
+"@artsy/cohesion@^4.233.0":
   version "4.233.0"
   resolved "https://registry.yarnpkg.com/@artsy/cohesion/-/cohesion-4.233.0.tgz#d2765064684006916918e8afde988cb0b5813b7f"
   integrity sha512-lNRi2cMecJsveuhLqeehePsibXxAtO4nPxn1wXS3z1pyhE49yN/OJ1HJycOoBLfEYZjDudLL7/MeD5MlJaJL9A==