Email preferences are defined in schema but not respected in email sending logic

[hetrakholiya11]

Team 101

Problem:
The User model includes an emailPreferences field in the Prisma schema, but the email sending logic does not check or respect these preferences before sending emails.

What’s happening:
In the database schema, users have configurable email preference flags such as:

• welcomeEmail
• streakReminder
• streakBroken
• weeklySummary

However, functions like:

• sendStreakReminder
• sendStreakBrokenNotification
• sendWeeklySummary
• sendWelcomeEmail

send emails without checking whether the user has opted out of that specific email type.

Why this is important:
This results in users receiving emails even if they have disabled that notification type in their preferences.

From a product perspective:

• This breaks expected user behavior
• Makes the emailPreferences field effectively unused
• Reduces trust in notification controls
• Creates an incomplete feature implementation

Expected behavior:
Before sending any email, the system should check the user's emailPreferences field and only send the email if the relevant preference is enabled.

Suggested improvement:

• Fetch user.emailPreferences before sending emails
• Gate each email type based on its corresponding preference flag
• Skip sending the email if the preference is set to false
• Optionally pass preferences into the email service to avoid extra DB queries

This would complete the existing email preference infrastructure and make the notification system behave as intended.

I would be happy to work on implementing this if assigned.

[RutviShah2]

T-149

We would like to work on this issue.
please assign us this issue

[DaxGondaliya]

Team: T163

I would like to work on this issue.

Proposed Solution (Short):

Before sending any email, I will add a preference check against user.emailPreferences.

Plan:

Fetch user.emailPreferences when preparing to send an email.
Gate each email function:
sendWelcomeEmail → check welcomeEmail
sendStreakReminder → check streakReminder
sendStreakBrokenNotification → check streakBroken
sendWeeklySummary → check weeklySummary

If the respective flag is false, skip sending the email.

Optionally refactor the email service to accept preferences as a parameter to avoid redundant DB queries.

This will ensure notification settings are respected and complete the intended email preference feature properly.

[HEETMEHTA18]

Team T143 Would like to work on this

[Aryan-B-Parikh]

I would like to solve this issue. Please assign it to us
Team:-146