Skip to content

Commit 7f2c576

Browse files
committed
fix email merge template issue
1 parent b124deb commit 7f2c576

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

framework/auth/views.py

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -832,7 +832,6 @@ def send_confirm_email(user, email, renew=False, external_id_provider=None, exte
832832
merge_target = OSFUser.objects.get(emails__address=email)
833833
except OSFUser.DoesNotExist:
834834
merge_target = None
835-
merge_account_data = {}
836835
campaign = campaigns.campaign_for_user(user)
837836
# Choose the appropriate email template to use and add existing_user flag if a merge or adding an email.
838837
if external_id_provider and external_id:
@@ -845,11 +844,6 @@ def send_confirm_email(user, email, renew=False, external_id_provider=None, exte
845844
raise HTTPError(http_status.HTTP_400_BAD_REQUEST, data={})
846845
elif merge_target:
847846
# Merge account confirmation
848-
merge_account_data = {
849-
'merge_target_fullname': merge_target.fullname or merge_target.username,
850-
'user_username': user.fullname,
851-
'email': merge_target.email,
852-
}
853847
notification_type = NotificationType.Type.USER_CONFIRM_MERGE
854848
elif user.is_active:
855849
# Add email confirmation
@@ -866,12 +860,14 @@ def send_confirm_email(user, email, renew=False, external_id_provider=None, exte
866860
subscribed_object=user,
867861
event_context={
868862
'user_fullname': user.fullname,
863+
'user_username': user.username,
869864
'confirmation_url': confirmation_url,
870865
'can_change_preferences': False,
871866
'external_id_provider': external_id_provider,
872867
'osf_contact_email': settings.OSF_CONTACT_EMAIL,
873868
'osf_support_email': settings.OSF_SUPPORT_EMAIL,
874-
**merge_account_data,
869+
'merge_target_fullname': merge_target.fullname or merge_target.username,
870+
'email': merge_target.email,
875871
},
876872
)
877873

0 commit comments

Comments
 (0)