Skip to content

Commit 569ebae

Browse files
committed
Backend: default notification email feature flags to off
The notification_translations_enabled and email_ics_attachments_enabled flags were defaulting to True, leaving both features on when the flag is absent. Default them to False to match every other feature gate.
1 parent cd48344 commit 569ebae

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

app/backend/src/couchers/notifications/background.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,14 +44,14 @@ def _send_email_notification(session: Session, user: User, notification: Notific
4444
context = make_background_user_context(user.id)
4545

4646
loc_context = LocalizationContext.from_user(user)
47-
if not context.get_boolean_value("notification_translations_enabled", default=True):
47+
if not context.get_boolean_value("notification_translations_enabled", default=False):
4848
loc_context = dataclasses.replace(loc_context, locale="en")
4949

5050
rendered = render_email_notification(
5151
user,
5252
notification,
5353
loc_context,
54-
include_ics_attachments=context.get_boolean_value("email_ics_attachments_enabled", default=True),
54+
include_ics_attachments=context.get_boolean_value("email_ics_attachments_enabled", default=False),
5555
)
5656

5757
queue_email(

0 commit comments

Comments
 (0)