diff --git a/templates/zerver/invalid_email.html b/templates/zerver/invalid_email.html index d84bde29c1..78e6aacecd 100644 --- a/templates/zerver/invalid_email.html +++ b/templates/zerver/invalid_email.html @@ -5,46 +5,38 @@ {% endblock %} {% block portico_content %} - -

{{ _('Invalid email') }}

- -

{{ _('Hi there! Thank you for your interest in Zulip.') }}

- - {% if invalid_email %} -

- {% trans %} - The email address you are trying to sign up with is not valid. - Please sign up using a valid email address. - {% endtrans %} -

- {% endif %} - - {% if closed_domain %} -

- {% trans %} - The organization you are trying to join, {{ realm_name }}, - only allows users with email addresses within the - organization. Please sign up using appropriate email address. - {% endtrans %} -

- {% endif %} - - {% if disposable_emails_not_allowed %} -

- {% trans %}The organization you are trying to join, - {{realm_name}}, does not allow signups using disposable email - addresses. Please sign up using a real email address. - {% endtrans %} -

- {% endif %} - - {% if email_contains_plus %} -

- {% trans %}The organization you are trying to join, - {{realm_name}}, does not allow signups using emails - that contains +. Please sign up using appropriate email address. - {% endtrans %} -

- {% endif %} - +
+
+
+
+ {% if invalid_email %} +

{{ _("Invalid email") }}

+ {% else %} +

{{ _("Email not allowed") }}

+ {% endif %} +
+
+

+ {% if invalid_email %} + {{ _("The email address you are trying to sign up with is not valid.") }} + {% endif %} + {% if closed_domain %} + {% trans %}The organization you are trying to join, {{ realm_name }}, does not allow signups using emails with your email domain.{% endtrans %} + {% endif %} + {% if disposable_emails_not_allowed %} + {% trans %}The organization you are trying to join, {{ realm_name }}, does not allow signups using disposable email addresses.{% endtrans %} + {% endif %} + {% if email_contains_plus %} + {% trans %}The organization you are trying to join, {{ realm_name }}, does not allow signups using emails that contain "+".{% endtrans %} + {% endif %} + {% if invalid_email %} + {{ _("Please sign up using a valid email address.") }} + {% else %} + {{ _("Please sign up using an allowed email address.") }} + {% endif %} +

+
+
+
+
{% endblock %} diff --git a/zerver/tests/test_invite.py b/zerver/tests/test_invite.py index b3ec2f722f..50ef91eb4d 100644 --- a/zerver/tests/test_invite.py +++ b/zerver/tests/test_invite.py @@ -1094,7 +1094,9 @@ so we didn't send them an invitation. We did send invitations to everyone else!" result = self.submit_reg_form_for_user("foo@example.com", "password") self.assertEqual(result.status_code, 200) - self.assert_in_response("only allows users with email addresses", result) + self.assert_in_response( + "does not allow signups using emails with your email domain", result + ) def test_disposable_emails_before_closing(self) -> None: """ @@ -1119,7 +1121,7 @@ so we didn't send them an invitation. We did send invitations to everyone else!" result = self.submit_reg_form_for_user("foo@mailnator.com", "password") self.assertEqual(result.status_code, 200) - self.assert_in_response("Please sign up using a real email address.", result) + self.assert_in_response("does not allow signups using disposable email addresses.", result) def test_invite_with_email_containing_plus_before_closing(self) -> None: """ @@ -1145,9 +1147,7 @@ so we didn't send them an invitation. We did send invitations to everyone else!" result = self.submit_reg_form_for_user(external_address, "password") self.assertEqual(result.status_code, 200) - self.assert_in_response( - "Zulip Dev, does not allow signups using emails\n that contains +", result - ) + self.assert_in_response('does not allow signups using emails that contain "+".', result) def test_invalid_email_check_after_confirming_email(self) -> None: self.login("hamlet")