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

Mephistic · web-flow · commit b6e19b9a6313 · 2025-08-06T18:48:44.000-04:00
Fallback to publicEmail for Org Users in Email notifications
diff --git a/functions/src/notifications/deliverNotifications.ts b/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`
diff --git a/functions/src/notifications/types.ts b/functions/src/notifications/types.ts
@@ -68,4 +68,7 @@ export interface Profile {
   email?: string
   notificationFrequency?: Frequency
   nextDigestAt?: FirebaseFirestore.Timestamp
+  contactInfo?: {
+    publicEmail?: string
+  }
 }

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ const deliverEmailNotifications = async () => {`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`// Temporarily using email from the profile to test the non-auth issues`
`69`		`- const verifiedEmail = profile.email //await getVerifiedUserEmail(profileDoc.id)`
	`69`	`+ const verifiedEmail = profile.email \|\| profile.contactInfo?.publicEmail //await getVerifiedUserEmail(profileDoc.id)`
`70`	`70`	`if (!verifiedEmail) {`
`71`	`71`	`console.log(`
`72`	`72`	`Skipping user ${profileDoc.id} because they have no verified email address`
Original file line number	Diff line number	Diff line change
`@@ -68,4 +68,7 @@ export interface Profile {`
`68`	`68`	`email?: string`
`69`	`69`	`notificationFrequency?: Frequency`
`70`	`70`	`nextDigestAt?: FirebaseFirestore.Timestamp`
	`71`	`+ contactInfo?: {`
	`72`	`+ publicEmail?: string`
	`73`	`+ }`
`71`	`74`	`}`