diff --git a/corporate/tests/test_remote_billing.py b/corporate/tests/test_remote_billing.py index 5a66b582b9..8d5a3dcb64 100644 --- a/corporate/tests/test_remote_billing.py +++ b/corporate/tests/test_remote_billing.py @@ -105,7 +105,7 @@ class RemoteBillingAuthenticationTest(BouncerTestCase): # Final confirmation page - just confirm your details, possibly # agreeing to ToS if needed and an authenticated session will be granted: self.assertEqual(result.status_code, 200) - self.assert_in_success_response(["Log in to Zulip server billing"], result) + self.assert_in_success_response(["Log in to Zulip plan management"], result) self.assert_in_success_response([user.realm.host], result) params = {} @@ -510,7 +510,7 @@ class LegacyServerLoginTest(BouncerTestCase): result = self.client_get(confirmation_url, subdomain="selfhosting") self.assertEqual(result.status_code, 200) self.assert_in_success_response( - [f"Log in to Zulip server billing for {self.server.hostname}", email], result + [f"Log in to Zulip plan management for {self.server.hostname}", email], result ) self.assert_in_success_response([f'action="{confirmation_url}"'], result) if expect_tos: diff --git a/corporate/views/remote_billing_page.py b/corporate/views/remote_billing_page.py index da15561397..7b2e665535 100644 --- a/corporate/views/remote_billing_page.py +++ b/corporate/views/remote_billing_page.py @@ -203,7 +203,7 @@ def remote_realm_billing_finalize_login( } return render( request, - "corporate/remote_realm_billing_finalize_login_confirmation.html", + "corporate/remote_billing_finalize_login_confirmation.html", context=context, ) else: @@ -218,9 +218,7 @@ def remote_realm_billing_finalize_login( } return render( request, - # TODO: We're re-using a template originally made for the legacy server - # flow. We should rename it and its HTML contents to a more general name. - "corporate/remote_billing_legacy_server_confirm_login_form.html", + "corporate/remote_billing_confirm_email_form.html", context=context, ) @@ -562,7 +560,7 @@ def remote_billing_legacy_server_login( } return render( request, - "corporate/remote_billing_legacy_server_confirm_login_form.html", + "corporate/remote_billing_confirm_email_form.html", context=context, ) @@ -677,9 +675,7 @@ def remote_billing_legacy_server_from_login_confirmation_link( } return render( request, - # TODO: We're re-using the template, so it should be renamed - # to a more general name. - "corporate/remote_realm_billing_finalize_login_confirmation.html", + "corporate/remote_billing_finalize_login_confirmation.html", context=context, ) diff --git a/templates/corporate/remote_billing_legacy_server_confirm_login_form.html b/templates/corporate/remote_billing_confirm_email_form.html similarity index 86% rename from templates/corporate/remote_billing_legacy_server_confirm_login_form.html rename to templates/corporate/remote_billing_confirm_email_form.html index 23ef1681bf..1644694348 100644 --- a/templates/corporate/remote_billing_legacy_server_confirm_login_form.html +++ b/templates/corporate/remote_billing_confirm_email_form.html @@ -14,7 +14,7 @@

Next, we will send a verification email to the address you provide.

-
+ {{ csrf_input }} {% if next_page %} @@ -24,7 +24,7 @@
- diff --git a/templates/corporate/remote_realm_billing_finalize_login_confirmation.html b/templates/corporate/remote_billing_finalize_login_confirmation.html similarity index 60% rename from templates/corporate/remote_realm_billing_finalize_login_confirmation.html rename to templates/corporate/remote_billing_finalize_login_confirmation.html index 9e47653e71..bc093c8493 100644 --- a/templates/corporate/remote_realm_billing_finalize_login_confirmation.html +++ b/templates/corporate/remote_billing_finalize_login_confirmation.html @@ -2,22 +2,21 @@ {% set entrypoint = "upgrade" %} {% block title %} -{{ _("Billing login confirmation") }} | Zulip +{{ _("Zulip plan management login confirmation") }} | Zulip {% endblock %} {% block portico_content %} -
+