fix: messages when recipient doesn't have auth_id (#4152)

src/routes/api.messages.tsx

@@ -44,6 +44,7 @@ export const action = async ({ request }: ActionFunctionArgs) => {
 
     const from = userProfile.data!.at(0)!.id
     const to = recipientProfile.data!.at(0)!.id
+    const toAuthId = recipientProfile.data!.at(0)!.auth_id
 
     const today = new Date()
     const yesterday = new Date(today.getTime() - 1000 * 60 * 60 * 24)
@@ -81,9 +82,12 @@ export const action = async ({ request }: ActionFunctionArgs) => {
       throw messageResult.error
     }
 
-    const emailResult = await client.rpc('get_user_email_by_id', {
-      id: recipientProfile.data![0].auth_id,
-    })
+    // TODO: use get_user_email_by_id only after removing firebase completely and all profiles have an auth_id
+    const emailResult = toAuthId
+      ? await client.rpc('get_user_email_by_id', { id: toAuthId })
+      : await client.rpc('get_user_email_by_username', {
+          username: data.to,
+        })
     const receiverEmail = emailResult.data[0].email
     const fromUsername = userProfile.data![0].username
     const emailTemplate = (

supabase/migrations/20250307061534_messages_temp_fix.sql

@@ -0,0 +1,10 @@
+CREATE OR REPLACE FUNCTION public.get_user_email_by_username(username text)
+ RETURNS TABLE(email character varying)
+ LANGUAGE plpgsql
+ SECURITY DEFINER
+AS $function$
+BEGIN
+  RETURN QUERY SELECT au.email FROM auth.users au WHERE au.raw_user_meta_data->>'username' = $1;
+END;
+$function$
+;