darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-24 00:07:02 -04:00
Code Issues Packages Projects Releases Wiki Activity

Make reply to address dependent on sending intent

Browse Source 
We have a number of go live requests where people have said they’re
sending email, but haven’t set up a reply-to address. At the moment we
don’t prompt them to, because we look at whether they have email
templates as indicative of whether they’re going to send email.

Now that we explicitly ask for the volumes of email they’re sending we
should use this to determine whether or not we prompt them to set up an
email reply to address because it’s a stronger signal of intent than
what templates they’ve set up.
This commit is contained in:
Chris Hill-Scott
2019-03-01 14:11:33 +00:00
parent 9bdcd81c65
commit 24ee280632
2 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/service.py
View File

@@ -226,7 +226,7 @@ class Service():
@property
def needs_to_add_email_reply_to_address(self):
return self.has_email_templates and not self.has_email_reply_to_address
return self.volume_email and not self.has_email_reply_to_address
@property
def shouldnt_use_govuk_as_sms_sender(self):