From 9c6eeeb4260f9e477d71adebe8633f23400ad8be Mon Sep 17 00:00:00 2001
From: Stephen Adamson <popertots@gmail.com>
Date: Sat, 6 Jun 2026 15:18:37 +0100
Subject: [PATCH] Fixes

---
 .../admin/BookingCreateModal.svelte           | 15 +++++--
 .../components/admin/WalkInCreateModal.svelte | 17 ++++---
 frontend/src/routes/account/+page.svelte      | 44 +++----------------
 3 files changed, 30 insertions(+), 46 deletions(-)

diff --git a/frontend/src/lib/components/admin/BookingCreateModal.svelte b/frontend/src/lib/components/admin/BookingCreateModal.svelte
index d4d88ce..c978d31 100644
--- a/frontend/src/lib/components/admin/BookingCreateModal.svelte
+++ b/frontend/src/lib/components/admin/BookingCreateModal.svelte
@@ -942,12 +942,19 @@
 									<input
 										id="guest-phone"
 										type="tel"
-										class="flex h-10 w-full rounded-md border border-input bg-background px-3 py-2 text-sm ring-offset-background focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 focus-visible:outline-none"
+										class="flex h-10 w-full rounded-md border bg-background px-3 py-2 text-sm ring-offset-background focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 focus-visible:outline-none {guestPhoneError
+											? 'border-red-500'
+											: 'border-input'}"
 										placeholder="07700 900000"
 										value={guestPhone}
-										oninput={(e) => { guestPhone = e.currentTarget.value; }}
-										onblur={() => { if (guestPhone && !isValidUKPhone(guestPhone)) guestPhoneError = 'Invalid UK phone number'; else guestPhoneError = ''; }}
-										class={guestPhoneError ? 'border-red-500' : ''}
+										oninput={(e) => {
+											guestPhone = e.currentTarget.value;
+										}}
+										onblur={() => {
+											if (guestPhone && !isValidUKPhone(guestPhone))
+												guestPhoneError = 'Invalid UK phone number';
+											else guestPhoneError = '';
+										}}
 									/>
 									{#if guestPhoneError}
 										<p class="mt-1 text-xs text-red-600">{guestPhoneError}</p>
diff --git a/frontend/src/lib/components/admin/WalkInCreateModal.svelte b/frontend/src/lib/components/admin/WalkInCreateModal.svelte
index c50bbc2..f5661cb 100644
--- a/frontend/src/lib/components/admin/WalkInCreateModal.svelte
+++ b/frontend/src/lib/components/admin/WalkInCreateModal.svelte
@@ -625,12 +625,19 @@
 									<input
 										id="guest-phone"
 										type="tel"
-										class="flex h-10 w-full rounded-md border border-input bg-background px-3 py-2 text-sm ring-offset-background focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 focus-visible:outline-none"
-										placeholder="07700 900000 (optional — helps us reach you if running late)"
+										class="flex h-10 w-full rounded-md border bg-background px-3 py-2 text-sm ring-offset-background focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 focus-visible:outline-none {guestPhoneError
+											? 'border-red-500'
+											: 'border-input'}"
+										placeholder="07700 900000"
 										value={guestPhone}
-										oninput={(e) => { guestPhone = e.currentTarget.value; }}
-										onblur={() => { guestPhoneError = guestPhone && !isValidUKPhone(guestPhone) ? 'Invalid UK phone number' : ''; }}
-										class={guestPhoneError ? 'border-red-500' : ''}
+										oninput={(e) => {
+											guestPhone = e.currentTarget.value;
+										}}
+										onblur={() => {
+											if (guestPhone && !isValidUKPhone(guestPhone))
+												guestPhoneError = 'Invalid UK phone number';
+											else guestPhoneError = '';
+										}}
 									/>
 									{#if guestPhoneError}
 										<p class="mt-1 text-xs text-red-600">{guestPhoneError}</p>
diff --git a/frontend/src/routes/account/+page.svelte b/frontend/src/routes/account/+page.svelte
index 8a933a3..e7c48b1 100644
--- a/frontend/src/routes/account/+page.svelte
+++ b/frontend/src/routes/account/+page.svelte
@@ -6,7 +6,7 @@
 	import { toast } from 'svelte-sonner';
 	import UserBookingModal from '$lib/components/account/UserBookingModal.svelte';
 	import { isValidUKPhone, formatPhoneDisplay, toE164UK } from '$lib/utils/phone';
-	import { savedCardsStore } from '$lib/stores/savedCards.svelte';
+	import { savedCardsStore, type SavedCard } from '$lib/stores/savedCards.svelte';
 
 	// zxcvbn-ts imports
 	import { zxcvbn, zxcvbnOptions } from '@zxcvbn-ts/core';
@@ -120,38 +120,8 @@
 
 	let notifPrefs = $state({ emailEnabled: true, smsEnabled: true, browserPushEnabled: true });
 
-	type SavedCard = {
-		id: string;
-		brand: string;
-		last_4: string;
-		exp_month: number;
-		exp_year: number;
-		is_default: boolean;
-	};
-
-	let savedCards = $state<SavedCard[]>([]);
 	let loadingCards = $state(false);
 
-	async function fetchSavedCards() {
-		loadingCards = true;
-		try {
-			const res = await fetch('/api/user/payment-methods', {
-				headers: { Authorization: `Bearer ${authStore.currentToken}` }
-			});
-			if (res.ok) {
-				savedCards = await res.json();
-				if (savedCards.length > 0 && !buySelectedCard) {
-					const defaultCard = savedCards.find((c) => c.is_default) || savedCards[0];
-					buySelectedCard = defaultCard.id;
-				}
-			}
-		} catch {
-			toast.error('Failed to load saved cards');
-		} finally {
-			loadingCards = false;
-		}
-	}
-
 	let cardToDelete = $state<SavedCard | null>(null);
 	let showDeleteCardDialog = $state(false);
 
@@ -163,7 +133,7 @@
 			});
 			if (res.ok) {
 				toast.success('Card removed');
-				savedCards = savedCards.filter((c) => c.id !== card.id);
+				savedCardsStore = savedCardsStore.filter((c) => c.id !== card.id);
 			} else {
 				toast.error('Failed to remove card');
 			}
@@ -444,7 +414,7 @@
 	}
 
 	$effect(() => {
-		if (savedCards.length === 0 && buySelectedCard !== '') {
+		if (savedCardsStore.length === 0 && buySelectedCard !== '') {
 			buySelectedCard = '';
 		}
 	});
@@ -1834,14 +1804,14 @@
 									</Button>
 								</div>
 							</div>
-						{:else if savedCards.length === 0}
+						{:else if savedCardsStore.length === 0}
 							<div class="py-8 text-center">
 								<p class="text-gray-500">No saved cards yet</p>
 								<Button class="mt-4" onclick={() => (showAddCard = true)}>Add a Card</Button>
 							</div>
 						{:else}
 							<div class="space-y-3">
-								{#each savedCards as card (card.id)}
+								{#each savedCardsStore as card (card.id)}
 									<div class="flex items-center justify-between rounded-lg border p-4">
 										<div class="flex items-center gap-3">
 											<div
@@ -2075,9 +2045,9 @@
 									<span class="block text-xs font-semibold tracking-wider text-gray-500 uppercase"
 										>Payment Method</span
 									>
-									{#if savedCards.length > 0}
+									{#if savedCardsStore.length > 0}
 										<div class="space-y-2">
-											{#each savedCards as card (card.id)}
+											{#each savedCardsStore as card (card.id)}
 												<button
 													type="button"
 													class="flex w-full items-center justify-between rounded-lg border p-3 text-left {buySelectedCard ===