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

fix: race condition with booking confirmation being set in session storage

Browse Source
This commit is contained in:
Tobias Johansson
2024-10-10 15:55:51 +02:00
parent 740419bad2
commit 1bce311666
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
components/HotelReservation/SelectRate/Payment/index.tsx
View File

@@ -58,11 +58,6 @@ export default function Payment({ hotel }: PaymentProps) {
const initiateBooking = trpc.booking.booking.create.useMutation({ const initiateBooking = trpc.booking.booking.create.useMutation({
onSuccess: (result) => { onSuccess: (result) => {
if (result?.confirmationNumber) { if (result?.confirmationNumber) {
// Planet doesn't support query params so we have to store values in session storage
sessionStorage.setItem(
BOOKING_CONFIRMATION_NUMBER,
result.confirmationNumber
)
setConfirmationNumber(result.confirmationNumber) setConfirmationNumber(result.confirmationNumber)
} else { } else {
// TODO: add proper error message // TODO: add proper error message
@@ -85,6 +80,8 @@ export default function Payment({ hotel }: PaymentProps) {
useEffect(() => { useEffect(() => {
if (confirmationNumber && bookingStatus?.data?.paymentUrl) { if (confirmationNumber && bookingStatus?.data?.paymentUrl) {
// Planet doesn't support query params so we have to store values in session storage
sessionStorage.setItem(BOOKING_CONFIRMATION_NUMBER, confirmationNumber)
router.push(bookingStatus.data.paymentUrl) router.push(bookingStatus.data.paymentUrl)
} }
}, [confirmationNumber, bookingStatus, router]) }, [confirmationNumber, bookingStatus, router])