Merge pull request #1886 from Mephistic/fix-org-email

Fallback to publicEmail for Org Users in Email notifications

Author: Mephistic

functions/src/notifications/deliverNotifications.ts

@@ -66,7 +66,7 @@ const deliverEmailNotifications = async () => {
     }
 
     // Temporarily using email from the profile to test the non-auth issues
-    const verifiedEmail = profile.email //await getVerifiedUserEmail(profileDoc.id)
+    const verifiedEmail = profile.email || profile.contactInfo?.publicEmail //await getVerifiedUserEmail(profileDoc.id)
     if (!verifiedEmail) {
       console.log(
         `Skipping user ${profileDoc.id} because they have no verified email address`

functions/src/notifications/types.ts

@@ -68,4 +68,7 @@ export interface Profile {
   email?: string
   notificationFrequency?: Frequency
   nextDigestAt?: FirebaseFirestore.Timestamp
+  contactInfo?: {
+    publicEmail?: string
+  }
 }