From 1b82afb6bbdacec152d33ae2e7000475b39e4293 Mon Sep 17 00:00:00 2001 From: Katie Smith Date: Mon, 18 Dec 2017 11:39:21 +0000 Subject: [PATCH] Change reply to address for invitation emails to be invitation sender If someone receives an invitation email for Notify the reply-to address of the email was the GOV.UK Notify email address. This has been changed to be the email address of the user who sent the invite. Pivotal story: https://www.pivotaltracker.com/story/show/153094646 --- app/invite/rest.py | 2 +- tests/app/invite/test_invite_rest.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/invite/rest.py b/app/invite/rest.py index 55cab2a30..ba263e1b3 100644 --- a/app/invite/rest.py +++ b/app/invite/rest.py @@ -42,7 +42,7 @@ def create_invited_user(service_id): notification_type=EMAIL_TYPE, api_key_id=None, key_type=KEY_TYPE_NORMAL, - reply_to_text=service.get_default_reply_to_email_address() + reply_to_text=invited_user.from_user.email_address ) send_notification_to_queue(saved_notification, False, queue=QueueNames.NOTIFY) diff --git a/tests/app/invite/test_invite_rest.py b/tests/app/invite/test_invite_rest.py index 1c4c6b10a..ebeeb3492 100644 --- a/tests/app/invite/test_invite_rest.py +++ b/tests/app/invite/test_invite_rest.py @@ -33,7 +33,7 @@ def test_create_invited_user(admin_request, sample_service, mocker, invitation_e assert json_resp['data']['id'] notification = Notification.query.first() - assert notification.reply_to_text == "notify@gov.uk" + assert notification.reply_to_text == invite_from.email_address mocked.assert_called_once_with([(str(notification.id))], queue="notify-internal-tasks")