Skip to content

Commit 2ad9313

Browse files
committed
In announcement tool, remove Reply-To default of
"ietf@ietf.org" and make field required. - Legacy-Id: 19155
1 parent 9a0d818 commit 2ad9313

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

ietf/secr/announcement/forms.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,14 +92,14 @@ def __init__(self, *args, **kwargs):
9292
self.fields['cc'].help_text = 'Use comma separated lists for emails (Cc, Bcc, Reply To)'
9393
self.fields['frm'].widget = forms.Select(choices=get_from_choices(user))
9494
self.fields['frm'].label = 'From'
95+
self.fields['reply_to'].required = True
9596
self.fields['nomcom'].label = 'NomCom message:'
9697
nomcom_roles = person.role_set.filter(group__in=self.fields['nomcom'].queryset,name='chair')
9798
secr_roles = person.role_set.filter(group__acronym='secretariat',name='secr')
9899
if nomcom_roles:
99100
self.initial['nomcom'] = nomcom_roles[0].group.pk
100101
if not nomcom_roles and not secr_roles:
101102
self.fields['nomcom'].widget = forms.HiddenInput()
102-
self.initial['reply_to'] = 'ietf@ietf.org'
103103

104104
if self.hidden:
105105
for key in list(self.fields.keys()):

ietf/secr/announcement/tests.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ def test_valid_submit(self):
9696
'to':'Other...',
9797
'to_custom':'rcross@amsl.com',
9898
'frm':'IETF Secretariat <ietf-secretariat@ietf.org>',
99+
'reply_to':'secretariat@ietf.org',
99100
'subject':'Test Subject',
100101
'body':'This is a test.'}
101102
self.client.login(username="secretary", password="secretary+password")

0 commit comments

Comments
 (0)