fix useUser always in loading state

BeatBuzzer · Dec 9, 2024 · e6b8fb7 · e6b8fb7
1 parent 4a50299
commit e6b8fb7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 14 deletions.
diff --git a/components/profile/ProfileInformation.vue b/components/profile/ProfileInformation.vue
@@ -6,7 +6,6 @@ const {user, loading, error, fetchUser} = useUser()
 onMounted(async () => {
   if (session.value) {
     await fetchUser()
-    console.log(user.value)
   }
 })
 

diff --git a/composables/useUser.ts b/composables/useUser.ts
@@ -1,23 +1,23 @@
 import type {GetUserResponse} from "@/types/api/users";
 
 export const useUser = () => {
-    const user = useState<GetUserResponse|null>('user', () => null)
+    const user = useState<GetUserResponse | null>('user', () => null)
     const loading = useState<boolean>('userLoading', () => false)
-    const error = useState<string|null>('userError', () => null)
+    const error = useState<string | null>('userError', () => null)
 
     const fetchUser = async () => {
-        // Only fetch if we don't have user data
-        if (!user.value) {
-            loading.value = true
-            error.value = null
-            try {
-                user.value = await useProfileInformation()
-            } catch (err) {
-                error.value = 'Failed to fetch user'
-                console.error(err)
-            } finally {
-                loading.value = false
+        try {
+            // Only set loading on initial fetch
+            if (!user.value) {
+                loading.value = true
             }
+
+            user.value = await useProfileInformation()
+        } catch (err) {
+            error.value = 'Failed to fetch user'
+            console.error(err)
+        } finally {
+            loading.value = false
         }
     }