Merged in fix/tracking-pageloadtime-fix (pull request #1247)

fix: using already created performance instead of creating a new one. Also payment tracking fix

* fix: using already created performance instead of creating a new one. Also payment tracking fix


Approved-by: Joakim Jäderberg
This commit is contained in:
Linus Flood
2025-02-04 09:46:58 +00:00
parent 1ed1e0a816
commit 7db6c54beb
2 changed files with 6 additions and 10 deletions

View File

@@ -49,8 +49,9 @@ export default function RouterTransition({
useEffect(() => { useEffect(() => {
if (!hasRun && !hasRunInitial.current) { if (!hasRun && !hasRunInitial.current) {
const perfObserver = new PerformanceObserver((observedEntries) => { if (performance) {
const entry = observedEntries.getEntriesByType("navigation")[0] const entry = performance.getEntriesByType("navigation")[0]
if (entry) { if (entry) {
const trackingData = { const trackingData = {
...pageData, ...pageData,
@@ -66,14 +67,9 @@ export default function RouterTransition({
hotelInfo: hotelInfo, hotelInfo: hotelInfo,
paymentInfo, paymentInfo,
}) })
perfObserver.disconnect()
} }
}) }
perfObserver.observe({
type: "navigation",
buffered: true,
})
hasRunInitial.current = true hasRunInitial.current = true
setHasRun() setHasRun()
} }

View File

@@ -129,7 +129,7 @@ export function trackPaymentEvent(paymentEvent: PaymentEvent) {
hotelId: paymentEvent.hotelId, hotelId: paymentEvent.hotelId,
}, },
paymentInfo: { paymentInfo: {
isSavedCreditCard: paymentEvent.isSavedCreditCard, isSavedCard: paymentEvent.isSavedCreditCard,
status: paymentEvent.status, status: paymentEvent.status,
type: paymentEvent.method, type: paymentEvent.method,
smsEnable: paymentEvent.smsEnable, smsEnable: paymentEvent.smsEnable,