Merge pull request #28359 from dimagi/nh/rep/notify_reqd

notify_addresses_str required, bump visibility
dimagi · Aug 12, 2020 · 9796055 · 9796055
2 parents 569189a + d38299d
commit 9796055
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/corehq/motech/forms.py b/corehq/motech/forms.py
@@ -48,7 +48,7 @@ class ConnectionSettingsForm(forms.ModelForm):
         label=_('Addresses to send notifications'),
         help_text=_('A comma-separated list of email addresses to send error '
                     'notifications'),
-        required=False,
+        required=True,
     )
 
     class Meta:
@@ -98,6 +98,7 @@ def __init__(self, domain, *args, **kwargs):
             crispy.Fieldset(
                 _('Remote API Connection'),
                 crispy.Field('name'),
+                crispy.Field('notify_addresses_str'),
                 crispy.Field('url'),
                 crispy.Field('auth_type'),
                 crispy.Field('api_auth_settings'),
@@ -106,7 +107,6 @@ def __init__(self, domain, *args, **kwargs):
                 crispy.Field('client_id'),
                 crispy.Field('plaintext_client_secret'),
                 twbscrispy.PrependedText('skip_cert_verify', ''),
-                crispy.Field('notify_addresses_str'),
                 self.test_connection_button,
             ),
             hqcrispy.FormActions(