feat: fix auth context, profile pic storage

AnkushSarkar10 · AnkushSarkar10 · commit 6164f3a957b6 · 2025-03-30T14:29:32.000-04:00
diff --git a/src/api/habits/firebase-mutations.ts b/src/api/habits/firebase-mutations.ts
@@ -10,10 +10,15 @@ import {
   updateDoc,
 } from 'firebase/firestore';
 
+import { getCurrentUserId } from '@/core/auth';
 import { type habitColors } from '@/ui/colors';
 
 import { db } from '../common/firebase';
-import { getUserById, type UserIdT } from '../users';
+import {
+  getUserById,
+  getUserWithRelationshipById,
+  type UserIdT,
+} from '../users';
 import {
   type DbHabitT,
   type DbParticipantT,
@@ -27,7 +32,9 @@ type ColorName = keyof typeof habitColors;
 export const createHabit = async (
   habitCreationInfo: HabitCreationT,
 ): Promise<DbHabitT> => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
+  const myId = getCurrentUserId();
+  const currentUserData = await getUserById(myId);
+  if (currentUserData == null) throw new Error('User data not found');
 
   const newHabit = {
     title: habitCreationInfo.title,
@@ -39,8 +46,8 @@ export const createHabit = async (
     },
     participants: {
       [myId]: {
-        displayName: 'Alex Chen', // TODO: Get from auth context
-        username: 'alexchen',
+        displayName: currentUserData.username,
+        username: currentUserData.username,
         lastActivity: serverTimestamp(),
         isOwner: true,
       } satisfies Omit<DbParticipantT, 'lastActivity'> & {
@@ -88,8 +95,8 @@ export const editHabit = async (
 };
 
 export const acceptHabitInvite = async (habitId: HabitIdT): Promise<void> => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
-  const me = await getUserById(myId);
+  const myId = getCurrentUserId();
+  const me = await getUserWithRelationshipById(myId);
   if (!me) throw new Error('User not found');
 
   const habitRef = doc(db, 'habits', habitId);
diff --git a/src/api/habits/use-habits.tsx b/src/api/habits/use-habits.tsx
@@ -1,6 +1,7 @@
 import { createQuery } from 'react-query-kit';
 
-import { type UserIdT } from '../users';
+import { getCurrentUserId } from '@/core';
+
 import { getHabits } from './firebase-queries';
 import { type HabitT } from './types';
 
@@ -10,7 +11,7 @@ type Variables = void;
 export const useHabits = createQuery<Response, Variables, Error>({
   queryKey: ['habits'],
   fetcher: async () => {
-    const myId = '1' as UserIdT; // TODO: Get from auth context
+    const myId = getCurrentUserId();
     return await getHabits(myId);
   },
 });
diff --git a/src/api/notifications/firebase-mutations.ts b/src/api/notifications/firebase-mutations.ts
@@ -6,6 +6,8 @@ import {
   serverTimestamp,
 } from 'firebase/firestore';
 
+import { getCurrentUserId } from '@/core';
+
 import { db } from '../common/firebase';
 import { type HabitIdT } from '../habits';
 import { type UserIdT } from '../users';
@@ -15,7 +17,7 @@ export const sendHabitInvite = async (
   habitId: HabitIdT,
   receiverId: UserIdT,
 ) => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
+  const myId = getCurrentUserId();
   const notification: Omit<DbHabitNotificationT, 'sentAt'> & {
     sentAt: ReturnType<typeof serverTimestamp>;
   } = {
@@ -29,7 +31,7 @@ export const sendHabitInvite = async (
 };
 
 export const sendNudge = async (habitId: HabitIdT, receiverId: UserIdT) => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
+  const myId = getCurrentUserId();
   const notification: Omit<DbHabitNotificationT, 'sentAt'> & {
     sentAt: ReturnType<typeof serverTimestamp>;
   } = {
diff --git a/src/api/notifications/use-notifications.tsx b/src/api/notifications/use-notifications.tsx
@@ -1,6 +1,7 @@
 import { createQuery } from 'react-query-kit';
 
-import { type UserIdT } from '../users';
+import { getCurrentUserId } from '@/core';
+
 import { getNotifications } from './firebase-queries';
 import { type FriendNotificationT, type HabitNotificationT } from './types';
 
@@ -10,7 +11,7 @@ type Variables = void;
 export const useNotifications = createQuery<Response, Variables, Error>({
   queryKey: ['notifications'],
   fetcher: async () => {
-    const myId = '1' as UserIdT; // TODO: Get from auth context
+    const myId = getCurrentUserId();
     return await getNotifications(myId);
   },
 });
diff --git a/src/api/users/firebase-mutations.ts b/src/api/users/firebase-mutations.ts
@@ -2,19 +2,19 @@ import {
   addDoc,
   collection,
   deleteDoc,
-  doc,
   getDocs,
   query,
   serverTimestamp,
-  setDoc,
   where,
 } from 'firebase/firestore';
 
+import { getCurrentUserId } from '@/core';
+
 import { db } from '../common/firebase';
 import { type DbRelationshipT, type UserIdT } from './types';
 
 export const sendFriendRequest = async (userId: UserIdT) => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
+  const myId = getCurrentUserId();
   await addDoc(collection(db, 'notifications'), {
     type: 'friendRequest',
     senderId: myId,
@@ -24,9 +24,11 @@ export const sendFriendRequest = async (userId: UserIdT) => {
 };
 
 export const acceptFriendRequest = async (userId: UserIdT) => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
-  
-  const relationship: Omit<DbRelationshipT, 'friendsSince'> & { friendsSince: ReturnType<typeof serverTimestamp> } = {
+  const myId = getCurrentUserId();
+
+  const relationship: Omit<DbRelationshipT, 'friendsSince'> & {
+    friendsSince: ReturnType<typeof serverTimestamp>;
+  } = {
     status: 'friends',
     friendsSince: serverTimestamp(),
   };
@@ -37,7 +39,7 @@ export const acceptFriendRequest = async (userId: UserIdT) => {
     userId2: userId,
     ...relationship,
   });
-  
+
   await addDoc(collection(db, 'relationships'), {
     userId1: userId,
     userId2: myId,
@@ -46,8 +48,8 @@ export const acceptFriendRequest = async (userId: UserIdT) => {
 };
 
 export const removeFriend = async (userId: UserIdT) => {
-  const myId = '1' as UserIdT; // TODO: Get from auth context
-  
+  const myId = getCurrentUserId();
+
   // Find and delete both relationship documents
   const relationshipsRef = collection(db, 'relationships');
   const q1 = query(
@@ -61,13 +63,10 @@ export const removeFriend = async (userId: UserIdT) => {
     where('userId2', '==', myId),
   );
 
-  const [snapshot1, snapshot2] = await Promise.all([
-    getDocs(q1),
-    getDocs(q2),
-  ]);
+  const [snapshot1, snapshot2] = await Promise.all([getDocs(q1), getDocs(q2)]);
 
   await Promise.all([
     ...snapshot1.docs.map((doc) => deleteDoc(doc.ref)),
     ...snapshot2.docs.map((doc) => deleteDoc(doc.ref)),
   ]);
-}; 
+};
diff --git a/src/api/users/firebase-queries.ts b/src/api/users/firebase-queries.ts
@@ -7,8 +7,11 @@ import {
   type Timestamp,
   where,
 } from 'firebase/firestore';
+import { getDownloadURL, ref } from 'firebase/storage';
 
-import { db } from '../common/firebase';
+import { getCurrentUserId } from '@/core';
+
+import { db, storage } from '../common/firebase';
 import {
   type DbRelationshipT,
   type DbUserT,
@@ -68,13 +71,19 @@ const getRelationshipStatus = async (
   return convertDbRelationshipToRelationship(relationship as DbRelationshipT);
 };
 
-export const getUserById = async (
+export const getUserById = async (id: UserIdT): Promise<UserT | null> => {
+  const userDoc = await getDoc(doc(db, 'users', id));
+  if (!userDoc.exists()) return null;
+  return convertDbUserToUser(id, userDoc.data() as DbUserT);
+};
+
+export const getUserWithRelationshipById = async (
   id: UserIdT,
 ): Promise<UserWithRelationshipT | null> => {
   const userDoc = await getDoc(doc(db, 'users', id));
   if (!userDoc.exists()) return null;
 
-  const myId = '1' as UserIdT; // TODO: Get from auth context
+  const myId = getCurrentUserId();
   const relationship = await getRelationshipStatus(myId, id);
 
   return {
@@ -141,9 +150,15 @@ export const searchUsers = async (searchQuery: string): Promise<UserT[]> => {
 };
 
 export const getUserPicture = async (userId: UserIdT): Promise<string> => {
-  const pictureDoc = await getDoc(doc(db, 'pictures', userId));
-  if (!pictureDoc.exists()) throw new Error('User picture not found');
-  return pictureDoc.data().url;
+  const defaultProfilePic = require('/assets/images/default_profile_pic.png');
+  try {
+    const pictureRef = ref(storage, `profilePics/${userId}`);
+    const url = await getDownloadURL(pictureRef);
+    return url;
+  } catch (error) {
+    console.log(`No profile image found for ${userId}, using default.`);
+    return defaultProfilePic;
+  }
 };
 
 export const getFriends = async (userId: UserIdT): Promise<UserT[]> => {
@@ -177,7 +192,7 @@ export const getFriends = async (userId: UserIdT): Promise<UserT[]> => {
   // Fetch user data for each friend
   const friends = await Promise.all(
     uniqueFriendIds.map(async (friendId) => {
-      const friend = await getUserById(friendId as UserIdT);
+      const friend = await getUserWithRelationshipById(friendId as UserIdT);
       if (!friend) throw new Error('Friend not found');
       return friend;
     }),
diff --git a/src/api/users/use-friends.tsx b/src/api/users/use-friends.tsx
@@ -1,7 +1,9 @@
 import { createQuery } from 'react-query-kit';
 
+import { getCurrentUserId } from '@/core';
+
 import { getFriends } from './firebase-queries';
-import { type UserIdT, type UserWithRelationshipT } from './types';
+import { type UserWithRelationshipT } from './types';
 
 type Response = UserWithRelationshipT[];
 type Variables = void;
@@ -10,7 +12,7 @@ type Variables = void;
 export const useFriends = createQuery<Response, Variables, Error>({
   queryKey: ['friends'],
   fetcher: async () => {
-    const myId = '1' as UserIdT; // TODO: Get from auth context
+    const myId = getCurrentUserId();
     const friends = await getFriends(myId);
     return friends.map((friend) => ({
       ...friend,
diff --git a/src/api/users/use-remove-friend.tsx b/src/api/users/use-remove-friend.tsx
@@ -2,7 +2,7 @@ import { createMutation } from 'react-query-kit';
 
 import { queryClient } from '../common';
 import { removeFriend } from './firebase-mutations';
-import { getUserById } from './firebase-queries';
+import { getUserWithRelationshipById } from './firebase-queries';
 import { type UserIdT, type UserT } from './types';
 
 type Variables = { id: UserIdT };
@@ -11,7 +11,7 @@ type Response = UserT;
 export const useRemoveFriend = createMutation<Response, Variables, Error>({
   mutationFn: async (variables) => {
     await removeFriend(variables.id);
-    const user = await getUserById(variables.id);
+    const user = await getUserWithRelationshipById(variables.id);
     if (!user) throw new Error('User not found');
     return user;
   },
diff --git a/src/api/users/use-send-friend-request.tsx b/src/api/users/use-send-friend-request.tsx
@@ -2,7 +2,7 @@ import { createMutation } from 'react-query-kit';
 
 import { queryClient } from '../common';
 import { sendFriendRequest } from './firebase-mutations';
-import { getUserById } from './firebase-queries';
+import { getUserWithRelationshipById } from './firebase-queries';
 import { type UserIdT, type UserT } from './types';
 
 type Variables = { id: UserIdT };
@@ -11,7 +11,7 @@ type Response = UserT;
 export const useSendFriendRequest = createMutation<Response, Variables, Error>({
   mutationFn: async (variables) => {
     await sendFriendRequest(variables.id);
-    const user = await getUserById(variables.id);
+    const user = await getUserWithRelationshipById(variables.id);
     if (!user) throw new Error('User not found');
     return user;
   },
diff --git a/src/api/users/use-user.tsx b/src/api/users/use-user.tsx
@@ -1,6 +1,6 @@
 import { createQuery } from 'react-query-kit';
 
-import { getUserById } from './firebase-queries';
+import { getUserWithRelationshipById } from './firebase-queries';
 import { type UserIdT, type UserWithRelationshipT } from './types';
 
 type Variables = { id: UserIdT };
@@ -9,7 +9,7 @@ type Response = UserWithRelationshipT;
 export const useUser = createQuery<Response, Variables, Error>({
   queryKey: ['user'],
   fetcher: async (variables) => {
-    const user = await getUserById(variables.id);
+    const user = await getUserWithRelationshipById(variables.id);
     if (!user) throw new Error('User not found');
     return user;
   },
diff --git a/src/app/auth/create-edit-profile.tsx b/src/app/auth/create-edit-profile.tsx
diff --git a/src/app/habits/view-habit.tsx b/src/app/habits/view-habit.tsx
diff --git a/src/components/habit-card.tsx b/src/components/habit-card.tsx
diff --git a/src/components/modify-habit-entry/modify-entry-modal.tsx b/src/components/modify-habit-entry/modify-entry-modal.tsx
diff --git a/src/core/auth/index.tsx b/src/core/auth/index.tsx
