scandic/web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merged in fix/tracking-multiple-times (pull request #1186)

Browse Source 
fix: avoid tracking multiple times on the same page

* fix: avoid tracking multiple times on the same page


Approved-by: Erik Tiekstra
This commit is contained in:
Linus Flood
2025-01-17 10:08:28 +00:00
parent 69b69af03c
commit d229b5c463
1 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
components/TrackingSDK/RouterTransition.tsx
View File

@@ -41,6 +41,8 @@ export default function RouterTransition({
const pathName = usePathname() const pathName = usePathname()
const { isTransitioning, stopRouterTransition } = useRouterTransitionStore() const { isTransitioning, stopRouterTransition } = useRouterTransitionStore()
const previousPathname = useRef<string | null>(null)
// We need this check to differentiate hard vs soft navigations // We need this check to differentiate hard vs soft navigations
// This is not because of StrictMode // This is not because of StrictMode
const hasRunInitial = useRef<boolean>(false) const hasRunInitial = useRef<boolean>(false)
@@ -112,13 +114,16 @@ export default function RouterTransition({
pageLoadTime: getPageLoadTime(), pageLoadTime: getPageLoadTime(),
} }
const pageObject = createSDKPageObject(trackingData) const pageObject = createSDKPageObject(trackingData)
pushToDataLayer({ if (previousPathname.current !== pathName) {
event: "pageView", pushToDataLayer({
pageInfo: pageObject, event: "pageView",
userInfo: userData, pageInfo: pageObject,
hotelInfo: hotelInfo, userInfo: userData,
paymentInfo, hotelInfo: hotelInfo,
}) paymentInfo,
})
}
previousPathname.current = pathName // Update for next render
} }
} }
}, [ }, [