From 0be329ff06b42c70e8504a08a2fc4f2b432995e6 Mon Sep 17 00:00:00 2001
From: Josh Creek <8179928+jcreek@users.noreply.github.com>
Date: Wed, 6 Nov 2024 17:49:52 +0000
Subject: [PATCH] refactor(#60): Utilise errors when signing out

---
 src/lib/components/SignOut.svelte | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/lib/components/SignOut.svelte b/src/lib/components/SignOut.svelte
index 9917635..1a9e5d2 100644
--- a/src/lib/components/SignOut.svelte
+++ b/src/lib/components/SignOut.svelte
@@ -1,6 +1,7 @@
 <script lang="ts">
 	import { createEventDispatcher, onDestroy } from 'svelte';
 	import { basket, type Basket, type Item } from '$lib/stores/basket.js';
+	import { logger } from '$lib/utils/logger.js';
 
 	const dispatch = createEventDispatcher();
 
@@ -19,8 +20,15 @@
 	onDestroy(unsubscribe);
 
 	async function signOut() {
-		// TODO use the error from the response
-		const { error } = await supabase.auth.signOut().then(() => {
+		const { error } = await supabase.auth.signOut();
+
+		if (error) {
+			logger.error('Error signing out:', error.message);
+			alert('Error signing out');
+			return;
+		}
+
+		try {
 			emitSignedOutEvent();
 
 			// Clear the basket
@@ -30,7 +38,10 @@
 			if (typeof localStorage !== 'undefined') {
 				localStorage.removeItem('basket');
 			}
-		});
+		} catch (err) {
+			logger.error('Error clearing basket:', err);
+			alert('Error clearing basket');
+		}
 	}
 </script>