From 06e571c225be8a6874e4a007f83afdf7cab4a7dd Mon Sep 17 00:00:00 2001 From: Anirudh Oppiliappan Date: Fri, 03 Jul 2026 10:13:55 +0000 Subject: [PATCH] appview/settings: show warning when no verified email for notifications --- appview/settings/settings.go | 8 ++++++-- appview/pages/templates/user/settings/notifications.html | 8 +++++++- 2 file(s) changed, 13 insertion(s)(+), 3 deletion(s)(-) diff --git a/appview/settings/settings.go b/appview/settings/settings.go --- a/appview/settings/settings.go +++ b/appview/settings/settings.go @@ -290,9 +290,13 @@ return } + primaryEmail, emailErr := db.GetPrimaryEmail(s.Db, did) + hasVerifiedEmail := emailErr == nil && primaryEmail.Verified + s.Pages.UserNotificationSettings(w, pages.UserNotificationSettingsParams{ - BaseParams: pages.BaseParamsFromContext(r.Context()), - Preferences: prefs, + BaseParams: pages.BaseParamsFromContext(r.Context()), + Preferences: prefs, + HasVerifiedEmail: hasVerifiedEmail, }) } diff --git a/appview/pages/templates/user/settings/notifications.html b/appview/pages/templates/user/settings/notifications.html --- a/appview/pages/templates/user/settings/notifications.html +++ b/appview/pages/templates/user/settings/notifications.html @@ -160,8 +160,14 @@
Email notifications
- Receive notifications via email in addition to in-app notifications. + Receive digest emails for issue and pull request activity.
+ {{ if not .HasVerifiedEmail }} +
+ {{ i "triangle-alert" "w-3.5 h-3.5 shrink-0" }} + You need a verified email address to receive email notifications. +
+ {{ end }}