From c063c1c5c45f68a018843ef2f21f7f29ac51642c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Roman=20Kr=C4=8Dek?= <roman.krcek@tietoevry.com>
Date: Tue, 24 Jun 2025 17:46:35 +0200
Subject: [PATCH] Improved UX for the creation flow

---
 src/routes/private/creator/+page.svelte       | 28 +++++++++++++------
 .../creator/steps/StepConnectGoogle.svelte    | 19 ++-----------
 .../private/creator/steps/StepOverview.svelte | 12 ++++----
 3 files changed, 27 insertions(+), 32 deletions(-)

diff --git a/src/routes/private/creator/+page.svelte b/src/routes/private/creator/+page.svelte
index 259fc29..4c1f3b1 100644
--- a/src/routes/private/creator/+page.svelte
+++ b/src/routes/private/creator/+page.svelte
@@ -11,9 +11,9 @@
   let participants = $state([]);
   let subject = $state('');
   let body = $state('');
+  let authorized = $state(false);
 
-  // Update events and participants from the form data
-  $effect( () => {
+  $effect(() => {
     if (form && form.new_event) {
       new_event = form.new_event;
     }
@@ -31,10 +31,15 @@
     StepOverview
   ];
 
-  // State variable for current step
-  let step = $state(0);
+  let step: number = $state(0);
+
+  let stepConditions = $derived([
+    authorized,
+    !!new_event?.name,
+    !!participants?.length,
+    !!subject && !!body
+  ]);
 
-  // Helper to go to next/previous step (optional)
   function nextStep() {
     if (step < steps.length - 1) step += 1;
   }
@@ -56,16 +61,15 @@
   </span>
   <button
     onclick={nextStep}
-    disabled={step === steps.length - 1}
+    disabled={step === steps.length - 1 || !stepConditions[step]}
     class="min-w-[100px] py-2 px-4 bg-white border border-gray-300 text-gray-700 rounded hover:bg-gray-50 disabled:opacity-50 disabled:cursor-not-allowed transition"
   >
     Next
   </button>
 </div>
 
-<!-- Render the current step component -->
 {#if step == 0}
-  <StepConnectGoogle />
+  <StepConnectGoogle bind:authorized />
 {:else if step == 1}
   <StepCreateEvent events={data.events} {new_event} />
 {:else if step == 2}
@@ -73,5 +77,11 @@
 {:else if step == 3}
   <StepCraftEmail bind:subject bind:body />
 {:else if step == 4}
-  <StepOverview {new_event} {participants} {subject} {body} />
+  <StepOverview
+    {new_event}
+    {participants}
+    {subject}
+    {body}
+    {stepConditions}
+  />
 {/if}
diff --git a/src/routes/private/creator/steps/StepConnectGoogle.svelte b/src/routes/private/creator/steps/StepConnectGoogle.svelte
index fddc9f7..77d8f99 100644
--- a/src/routes/private/creator/steps/StepConnectGoogle.svelte
+++ b/src/routes/private/creator/steps/StepConnectGoogle.svelte
@@ -2,8 +2,9 @@
 	import { onMount } from 'svelte';
 	import { goto } from '$app/navigation';
 
+	export let authorized = false;
+
 	let refreshToken = '';
-	let authorized = false;
 	let loading = true;
 
 	let to = '';
@@ -32,22 +33,6 @@
 	/* ⇢ redirects straight to Google via server 302 */
 	const connect = () => goto('/private/api/gmail?action=auth');
 
-	async function sendEmail() {
-		const r = await fetch('/private/api/gmail', {
-			method: 'POST',
-			headers: { 'Content-Type': 'application/json' },
-			body: JSON.stringify({
-				action: 'send',
-				to,
-				subject,
-				text: body,
-				refreshToken
-			})
-		});
-		r.ok ? alert('Sent!') : alert(await r.text());
-		to = subject = body = '';
-	}
-
 	async function disconnect() {
 		if (!confirm('Disconnect Google account?')) return;
 		await fetch('/private/api/gmail', {
diff --git a/src/routes/private/creator/steps/StepOverview.svelte b/src/routes/private/creator/steps/StepOverview.svelte
index 34cb497..2f68806 100644
--- a/src/routes/private/creator/steps/StepOverview.svelte
+++ b/src/routes/private/creator/steps/StepOverview.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-	let { new_event, participants, subject, body } = $props();
+	let { new_event, participants, subject, body, stepConditions } = $props();
 </script>
 
 <!-- New Event Overview -->
@@ -41,22 +41,22 @@
 	class="mt-4 w-full rounded bg-blue-600 px-4 py-3 font-bold text-white
     transition-colors duration-200 hover:bg-blue-700
     disabled:cursor-not-allowed disabled:bg-gray-300 disabled:text-gray-500"
-	disabled={!new_event?.name || !participants?.length || !subject || !body}
+	disabled={!stepConditions.every(Boolean)}
 >
 	Generate QR codes and send
 </button>
 
 <div class="mt-2 space-y-1">
-	{#if !new_event?.name}
+	{#if !stepConditions[0]}
 		<p class="text-sm text-red-500">Please provide an event name before proceeding.</p>
 	{/if}
-	{#if !participants?.length}
+	{#if !stepConditions[1]}
 		<p class="text-sm text-red-500">Please add at least one participant before proceeding.</p>
 	{/if}
-	{#if !subject}
+	{#if !stepConditions[2]}
 		<p class="text-sm text-red-500">Please provide an email subject before proceeding.</p>
 	{/if}
-	{#if !body}
+	{#if !stepConditions[3]}
 		<p class="text-sm text-red-500">Please provide an email body before proceeding.</p>
 	{/if}
 </div>