darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-02 17:31:14 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1504 from alphagov/change-reply-to-address-for-invites

Browse Source 
Change reply to address for invitation emails to be invitation sender
This commit is contained in:
Alexey Bezhan
2017-12-18 16:11:35 +00:00
committed by GitHub
parent 3015d4fc07 1b82afb6bb
commit 8e2f2818d7
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/invite/rest.py
View File

@@ -42,7 +42,7 @@ def create_invited_user(service_id):
notification_type=EMAIL_TYPE, notification_type=EMAIL_TYPE,
api_key_id=None, api_key_id=None,
key_type=KEY_TYPE_NORMAL, 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) send_notification_to_queue(saved_notification, False, queue=QueueNames.NOTIFY)

2
tests/app/invite/test_invite_rest.py
View File

@@ -33,7 +33,7 @@ def test_create_invited_user(admin_request, sample_service, mocker, invitation_e
assert json_resp['data']['id'] assert json_resp['data']['id']
notification = Notification.query.first() 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") mocked.assert_called_once_with([(str(notification.id))], queue="notify-internal-tasks")