Merge branch 'main' into za/1120-update-domain-overview-verbiage

This commit is contained in:
zandercymatics 2023-10-10 13:41:20 -06:00
commit 2129d2dd67
No known key found for this signature in database
GPG key ID: FF4636ABEC9682B7
7 changed files with 23 additions and 22 deletions

View file

@ -811,7 +811,8 @@ class DomainAdmin(ListHeaderAdmin):
else: else:
self.message_user( self.message_user(
request, request,
("Domain statuses are %s" ". Thanks!") % statuses, f"The registry statuses are {statuses}. "
"These statuses are from the provider of the .gov registry.",
) )
return HttpResponseRedirect(".") return HttpResponseRedirect(".")

View file

@ -13,10 +13,10 @@
{% elif original.state == original.State.ON_HOLD %} {% elif original.state == original.State.ON_HOLD %}
<input type="submit" value="Remove hold" name="_remove_client_hold"> <input type="submit" value="Remove hold" name="_remove_client_hold">
{% endif %} {% endif %}
<input id="manageDomainSubmitButton" type="submit" value="Manage Domain" name="_edit_domain"> <input id="manageDomainSubmitButton" type="submit" value="Manage domain" name="_edit_domain">
<input type="submit" value="get status" name="_get_status"> <input type="submit" value="Get registry status" name="_get_status">
{% if original.state != original.State.DELETED %} {% if original.state != original.State.DELETED %}
<input type="submit" value="Delete Domain in Registry" name="_delete_domain"> <input type="submit" value="Delete domain in registry" name="_delete_domain">
{% endif %} {% endif %}
</div> </div>
{{ block.super }} {{ block.super }}

View file

@ -1,7 +1,7 @@
{% extends "domain_base.html" %} {% extends "domain_base.html" %}
{% load static field_helpers%} {% load static field_helpers%}
{% block title %}Domain name servers | {{ domain.name }} | {% endblock %} {% block title %}DNS name servers | {{ domain.name }} | {% endblock %}
{% block domain_content %} {% block domain_content %}
{# this is right after the messages block in the parent template #} {# this is right after the messages block in the parent template #}
@ -9,7 +9,7 @@
{% include "includes/form_errors.html" with form=form %} {% include "includes/form_errors.html" with form=form %}
{% endfor %} {% endfor %}
<h1>Domain name servers</h1> <h1>DNS name servers</h1>
<p>Before your domain can be used we'll need information about your domain <p>Before your domain can be used we'll need information about your domain
name servers.</p> name servers.</p>

View file

@ -1,11 +1,11 @@
{% extends "domain_base.html" %} {% extends "domain_base.html" %}
{% load static field_helpers url_helpers %} {% load static field_helpers url_helpers %}
{% block title %}Domain security email | {{ domain.name }} | {% endblock %} {% block title %}Security email | {{ domain.name }} | {% endblock %}
{% block domain_content %} {% block domain_content %}
<h1>Domain security email</h1> <h1>Security email</h1>
<p>We strongly recommend that you provide a security email. This email will allow the public to report observed or suspected security issues on your domain. Security emails are made public and included in the <a href="{% public_site_url 'about/data/' %}">.gov domain data</a> we provide.</p> <p>We strongly recommend that you provide a security email. This email will allow the public to report observed or suspected security issues on your domain. Security emails are made public and included in the <a href="{% public_site_url 'about/data/' %}">.gov domain data</a> we provide.</p>

View file

@ -1,11 +1,11 @@
{% extends "domain_base.html" %} {% extends "domain_base.html" %}
{% load static field_helpers %} {% load static field_helpers %}
{% block title %}Domain contact information | {{ domain.name }} | {% endblock %} {% block title %}Your contact information | {{ domain.name }} | {% endblock %}
{% block domain_content %} {% block domain_content %}
<h1>Domain contact information</h1> <h1>Your contact information</h1>
<p>If youd like us to use a different name, email, or phone number you can make those changes below. <strong>Updating your contact information here will update the contact information for all domains in your account.</strong> However, it wont affect your Login.gov account information. <p>If youd like us to use a different name, email, or phone number you can make those changes below. <strong>Updating your contact information here will update the contact information for all domains in your account.</strong> However, it wont affect your Login.gov account information.
</p> </p>

View file

@ -109,12 +109,12 @@ class TestDomainAdmin(MockEppLib):
) )
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertContains(response, domain.name) self.assertContains(response, domain.name)
self.assertContains(response, "Delete Domain in Registry") self.assertContains(response, "Delete domain in registry")
# Test the info dialog # Test the info dialog
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Delete Domain in Registry", "name": domain.name}, {"_delete_domain": "Delete domain in registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client
@ -149,12 +149,12 @@ class TestDomainAdmin(MockEppLib):
) )
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertContains(response, domain.name) self.assertContains(response, domain.name)
self.assertContains(response, "Delete Domain in Registry") self.assertContains(response, "Delete domain in registry")
# Test the error # Test the error
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Delete Domain in Registry", "name": domain.name}, {"_delete_domain": "Delete domain in registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client
@ -194,12 +194,12 @@ class TestDomainAdmin(MockEppLib):
) )
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertContains(response, domain.name) self.assertContains(response, domain.name)
self.assertContains(response, "Delete Domain in Registry") self.assertContains(response, "Delete domain in registry")
# Test the info dialog # Test the info dialog
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Delete Domain in Registry", "name": domain.name}, {"_delete_domain": "Delete domain in registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client
@ -221,7 +221,7 @@ class TestDomainAdmin(MockEppLib):
# Test the info dialog # Test the info dialog
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Delete Domain in Registry", "name": domain.name}, {"_delete_domain": "Delete domain in registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client

View file

@ -1309,7 +1309,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib):
page = self.client.get( page = self.client.get(
reverse("domain-nameservers", kwargs={"pk": self.domain.id}) reverse("domain-nameservers", kwargs={"pk": self.domain.id})
) )
self.assertContains(page, "Domain name servers") self.assertContains(page, "DNS name servers")
@skip("Broken by adding registry connection fix in ticket 848") @skip("Broken by adding registry connection fix in ticket 848")
def test_domain_nameservers_form(self): def test_domain_nameservers_form(self):
@ -1414,7 +1414,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib):
page = self.client.get( page = self.client.get(
reverse("domain-your-contact-information", kwargs={"pk": self.domain.id}) reverse("domain-your-contact-information", kwargs={"pk": self.domain.id})
) )
self.assertContains(page, "Domain contact information") self.assertContains(page, "Your contact information")
def test_domain_your_contact_information_content(self): def test_domain_your_contact_information_content(self):
"""Logged-in user's contact information appears on the page.""" """Logged-in user's contact information appears on the page."""
@ -1439,7 +1439,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib):
) )
# Loads correctly # Loads correctly
self.assertContains(page, "Domain security email") self.assertContains(page, "Security email")
self.assertContains(page, "security@mail.gov") self.assertContains(page, "security@mail.gov")
self.mockSendPatch.stop() self.mockSendPatch.stop()
@ -1455,7 +1455,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib):
) )
# Loads correctly # Loads correctly
self.assertContains(page, "Domain security email") self.assertContains(page, "Security email")
self.assertNotContains(page, "dotgov@cisa.dhs.gov") self.assertNotContains(page, "dotgov@cisa.dhs.gov")
self.mockSendPatch.stop() self.mockSendPatch.stop()
@ -1464,7 +1464,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib):
page = self.client.get( page = self.client.get(
reverse("domain-security-email", kwargs={"pk": self.domain.id}) reverse("domain-security-email", kwargs={"pk": self.domain.id})
) )
self.assertContains(page, "Domain security email") self.assertContains(page, "Security email")
def test_domain_security_email_form(self): def test_domain_security_email_form(self):
"""Adding a security email works. """Adding a security email works.