From 259d83a71e5b7699416951d18eb574612c795869 Mon Sep 17 00:00:00 2001
From: Josh Creek <8179928+jcreek@users.noreply.github.com>
Date: Sat, 2 Nov 2024 15:32:52 +0000
Subject: [PATCH] refactor(#55): Improve error handling for
 requestAccountDeletion

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
---
 src/routes/account/+page.server.ts | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/routes/account/+page.server.ts b/src/routes/account/+page.server.ts
index 9e420a9..45ef748 100644
--- a/src/routes/account/+page.server.ts
+++ b/src/routes/account/+page.server.ts
@@ -23,8 +23,19 @@ export const actions: Actions = {
 		const { session } = await safeGetSession();
 		const baseUrl = url.origin;
 		if (session) {
-			await requestAccountDeletion(session.user.id, session.user.email!, baseUrl);
-			return {};
+			if (!session.user.email) {
+				return fail(400, { message: 'Email is required for account deletion' });
+			}
+			try {
+				await requestAccountDeletion(session.user.id, session.user.email, baseUrl);
+				return {
+					success: true,
+					message: 'Account deletion request submitted successfully'
+				};
+			} catch (error) {
+				console.error('Failed to request account deletion:', error);
+				return fail(500, { message: 'Failed to process deletion request' });
+			}
 		} else {
 			redirect(303, '/');
 		}