diff --git a/src/registrar/assets/sass/_theme/_buttons.scss b/src/registrar/assets/sass/_theme/_buttons.scss index d431bfa41..2e4469e12 100644 --- a/src/registrar/assets/sass/_theme/_buttons.scss +++ b/src/registrar/assets/sass/_theme/_buttons.scss @@ -1,7 +1,8 @@ @use "uswds-core" as *; @use "cisa_colors" as *; -/* Make "placeholder" links visually obvious */ +// Used on: TODO links +// Used on: NONE a[href$="todo"]::after { background-color: yellow; color: color(blue-80v); @@ -9,10 +10,14 @@ a[href$="todo"]::after { font-style: italic; } +// Used on: profile +// Note: Is this needed? a.usa-link.usa-link--always-blue { color: #{$dhs-blue}; } +// Used on: breadcrumbs +// Note: This could potentially be simplified and use usa-button--with-icon a.breadcrumb__back { display:flex; align-items: center; @@ -28,10 +33,18 @@ a.breadcrumb__back { } } +// Remove anchor buttons' underline a.usa-button { text-decoration: none; } +// Unstyled anchor buttons +a.usa-button--unstyled:visited { + color: color('primary'); +} + +// Disabled anchor buttons +// NOTE: Not used a.usa-button.disabled-link { background-color: #ccc !important; color: #454545 !important @@ -58,6 +71,8 @@ a.usa-button--unstyled.disabled-link:focus { text-decoration: none !important; } +// Disabled buttons +// Used on: Domain managers, disabled logo on profile .usa-button--unstyled.disabled-button, .usa-button--unstyled.disabled-button:hover, .usa-button--unstyled.disabled-button:focus { @@ -66,6 +81,16 @@ a.usa-button--unstyled.disabled-link:focus { text-decoration: none !important; } +// Unstyled variant for reverse out? +// Used on: NONE +.usa-button--unstyled--white, +.usa-button--unstyled--white:hover, +.usa-button--unstyled--white:focus, +.usa-button--unstyled--white:active { + color: color('white'); +} + +// Solid anchor buttons a.usa-button:not(.usa-button--unstyled, .usa-button--outline) { color: color('white'); } @@ -76,7 +101,8 @@ a.usa-button:not(.usa-button--unstyled, .usa-button--outline):focus, a.usa-button:not(.usa-button--unstyled, .usa-button--outline):active { color: color('white'); } - + +// Outline anchor buttons a.usa-button--outline, a.usa-button--outline:visited { box-shadow: inset 0 0 0 2px color('primary'); @@ -94,10 +120,22 @@ a.usa-button--outline:active { color: color('primary-darker'); } +// Used on: Domain request withdraw confirmation a.withdraw { background-color: color('error'); } + +a.withdraw:hover, +a.withdraw:focus { + background-color: color('error-dark'); +} +a.withdraw:active { + background-color: color('error-darker'); +} + +// Used on: Domain request status +//NOTE: Revise to BEM convention usa-button--outline-secondary a.withdraw_outline, a.withdraw_outline:visited { box-shadow: inset 0 0 0 2px color('error'); @@ -115,19 +153,8 @@ a.withdraw_outline:active { color: color('error-darker'); } -a.withdraw:hover, -a.withdraw:focus { - background-color: color('error-dark'); -} - -a.withdraw:active { - background-color: color('error-darker'); -} -a.usa-button--unstyled:visited { - color: color('primary'); -} - +// Used on: Domain request submit .dotgov-button--green { background-color: color('success-dark'); @@ -140,15 +167,8 @@ a.usa-button--unstyled:visited { } } -.usa-button--unstyled--white, -.usa-button--unstyled--white:hover, -.usa-button--unstyled--white:focus, -.usa-button--unstyled--white:active { - color: color('white'); -} - -// Cancel button used on the -// DNSSEC main page +// Cancel button +// Used on: DNSSEC main page // We want to center this button on mobile // and add some extra left margin on tablet+ .usa-button--cancel { @@ -175,6 +195,8 @@ a.usa-button--unstyled:visited { } } +// Used on: Profile page, toggleable fields +// Note: Could potentially be cleaned up by using usa-button--with-icon // We need to deviate from some default USWDS styles here // in this particular case, so we have to override this. .usa-form .usa-button.readonly-edit-button { @@ -186,6 +208,7 @@ a.usa-button--unstyled:visited { } } +//Used on: Domains and Requests tables .usa-button--filter { width: auto; // For mobile stacking @@ -201,6 +224,8 @@ a.usa-button--unstyled:visited { } } +// Buttons with nested icons +// Note: Can be simplified by adding usa-link--icon to anchors in tables .dotgov-table a, .usa-link--icon, .usa-button--with-icon { @@ -224,6 +249,9 @@ a .usa-icon, width: 1.5em; } +// Red, for delete buttons +// Used on: All delete buttons +// Note: Can be simplified by adding text-secondary to delete anchors in tables button.text-secondary, button.text-secondary:hover, .dotgov-table a.text-secondary { diff --git a/src/registrar/assets/sass/_theme/_forms.scss b/src/registrar/assets/sass/_theme/_forms.scss index 0aedfcdba..44c224aad 100644 --- a/src/registrar/assets/sass/_theme/_forms.scss +++ b/src/registrar/assets/sass/_theme/_forms.scss @@ -85,7 +85,7 @@ legend.float-left-tablet + button.float-right-tablet { .read-only-label { font-size: size('body', 'sm'); - color: color('primary'); + color: color('primary-dark'); margin-bottom: units(0.5); } diff --git a/src/registrar/migrations/0128_alter_domaininformation_state_territory_and_more.py b/src/registrar/migrations/0128_alter_domaininformation_state_territory_and_more.py new file mode 100644 index 000000000..a6b8d0519 --- /dev/null +++ b/src/registrar/migrations/0128_alter_domaininformation_state_territory_and_more.py @@ -0,0 +1,229 @@ +# Generated by Django 4.2.10 on 2024-09-11 21:00 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ("registrar", "0127_remove_domaininformation_submitter_and_more"), + ] + + operations = [ + migrations.AlterField( + model_name="domaininformation", + name="state_territory", + field=models.CharField( + blank=True, + choices=[ + ("AL", "Alabama (AL)"), + ("AK", "Alaska (AK)"), + ("AS", "American Samoa (AS)"), + ("AZ", "Arizona (AZ)"), + ("AR", "Arkansas (AR)"), + ("CA", "California (CA)"), + ("CO", "Colorado (CO)"), + ("CT", "Connecticut (CT)"), + ("DE", "Delaware (DE)"), + ("DC", "District of Columbia (DC)"), + ("FL", "Florida (FL)"), + ("GA", "Georgia (GA)"), + ("GU", "Guam (GU)"), + ("HI", "Hawaii (HI)"), + ("ID", "Idaho (ID)"), + ("IL", "Illinois (IL)"), + ("IN", "Indiana (IN)"), + ("IA", "Iowa (IA)"), + ("KS", "Kansas (KS)"), + ("KY", "Kentucky (KY)"), + ("LA", "Louisiana (LA)"), + ("ME", "Maine (ME)"), + ("MD", "Maryland (MD)"), + ("MA", "Massachusetts (MA)"), + ("MI", "Michigan (MI)"), + ("MN", "Minnesota (MN)"), + ("MS", "Mississippi (MS)"), + ("MO", "Missouri (MO)"), + ("MT", "Montana (MT)"), + ("NE", "Nebraska (NE)"), + ("NV", "Nevada (NV)"), + ("NH", "New Hampshire (NH)"), + ("NJ", "New Jersey (NJ)"), + ("NM", "New Mexico (NM)"), + ("NY", "New York (NY)"), + ("NC", "North Carolina (NC)"), + ("ND", "North Dakota (ND)"), + ("MP", "Northern Mariana Islands (MP)"), + ("OH", "Ohio (OH)"), + ("OK", "Oklahoma (OK)"), + ("OR", "Oregon (OR)"), + ("PA", "Pennsylvania (PA)"), + ("PR", "Puerto Rico (PR)"), + ("RI", "Rhode Island (RI)"), + ("SC", "South Carolina (SC)"), + ("SD", "South Dakota (SD)"), + ("TN", "Tennessee (TN)"), + ("TX", "Texas (TX)"), + ("UM", "United States Minor Outlying Islands (UM)"), + ("UT", "Utah (UT)"), + ("VT", "Vermont (VT)"), + ("VI", "Virgin Islands (VI)"), + ("VA", "Virginia (VA)"), + ("WA", "Washington (WA)"), + ("WV", "West Virginia (WV)"), + ("WI", "Wisconsin (WI)"), + ("WY", "Wyoming (WY)"), + ("AA", "Armed Forces Americas (AA)"), + ("AE", "Armed Forces Africa, Canada, Europe, Middle East (AE)"), + ("AP", "Armed Forces Pacific (AP)"), + ], + max_length=2, + null=True, + verbose_name="state, territory, or military post", + ), + ), + migrations.AlterField( + model_name="domainrequest", + name="state_territory", + field=models.CharField( + blank=True, + choices=[ + ("AL", "Alabama (AL)"), + ("AK", "Alaska (AK)"), + ("AS", "American Samoa (AS)"), + ("AZ", "Arizona (AZ)"), + ("AR", "Arkansas (AR)"), + ("CA", "California (CA)"), + ("CO", "Colorado (CO)"), + ("CT", "Connecticut (CT)"), + ("DE", "Delaware (DE)"), + ("DC", "District of Columbia (DC)"), + ("FL", "Florida (FL)"), + ("GA", "Georgia (GA)"), + ("GU", "Guam (GU)"), + ("HI", "Hawaii (HI)"), + ("ID", "Idaho (ID)"), + ("IL", "Illinois (IL)"), + ("IN", "Indiana (IN)"), + ("IA", "Iowa (IA)"), + ("KS", "Kansas (KS)"), + ("KY", "Kentucky (KY)"), + ("LA", "Louisiana (LA)"), + ("ME", "Maine (ME)"), + ("MD", "Maryland (MD)"), + ("MA", "Massachusetts (MA)"), + ("MI", "Michigan (MI)"), + ("MN", "Minnesota (MN)"), + ("MS", "Mississippi (MS)"), + ("MO", "Missouri (MO)"), + ("MT", "Montana (MT)"), + ("NE", "Nebraska (NE)"), + ("NV", "Nevada (NV)"), + ("NH", "New Hampshire (NH)"), + ("NJ", "New Jersey (NJ)"), + ("NM", "New Mexico (NM)"), + ("NY", "New York (NY)"), + ("NC", "North Carolina (NC)"), + ("ND", "North Dakota (ND)"), + ("MP", "Northern Mariana Islands (MP)"), + ("OH", "Ohio (OH)"), + ("OK", "Oklahoma (OK)"), + ("OR", "Oregon (OR)"), + ("PA", "Pennsylvania (PA)"), + ("PR", "Puerto Rico (PR)"), + ("RI", "Rhode Island (RI)"), + ("SC", "South Carolina (SC)"), + ("SD", "South Dakota (SD)"), + ("TN", "Tennessee (TN)"), + ("TX", "Texas (TX)"), + ("UM", "United States Minor Outlying Islands (UM)"), + ("UT", "Utah (UT)"), + ("VT", "Vermont (VT)"), + ("VI", "Virgin Islands (VI)"), + ("VA", "Virginia (VA)"), + ("WA", "Washington (WA)"), + ("WV", "West Virginia (WV)"), + ("WI", "Wisconsin (WI)"), + ("WY", "Wyoming (WY)"), + ("AA", "Armed Forces Americas (AA)"), + ("AE", "Armed Forces Africa, Canada, Europe, Middle East (AE)"), + ("AP", "Armed Forces Pacific (AP)"), + ], + max_length=2, + null=True, + verbose_name="state, territory, or military post", + ), + ), + migrations.AlterField( + model_name="portfolio", + name="state_territory", + field=models.CharField( + blank=True, + choices=[ + ("AL", "Alabama (AL)"), + ("AK", "Alaska (AK)"), + ("AS", "American Samoa (AS)"), + ("AZ", "Arizona (AZ)"), + ("AR", "Arkansas (AR)"), + ("CA", "California (CA)"), + ("CO", "Colorado (CO)"), + ("CT", "Connecticut (CT)"), + ("DE", "Delaware (DE)"), + ("DC", "District of Columbia (DC)"), + ("FL", "Florida (FL)"), + ("GA", "Georgia (GA)"), + ("GU", "Guam (GU)"), + ("HI", "Hawaii (HI)"), + ("ID", "Idaho (ID)"), + ("IL", "Illinois (IL)"), + ("IN", "Indiana (IN)"), + ("IA", "Iowa (IA)"), + ("KS", "Kansas (KS)"), + ("KY", "Kentucky (KY)"), + ("LA", "Louisiana (LA)"), + ("ME", "Maine (ME)"), + ("MD", "Maryland (MD)"), + ("MA", "Massachusetts (MA)"), + ("MI", "Michigan (MI)"), + ("MN", "Minnesota (MN)"), + ("MS", "Mississippi (MS)"), + ("MO", "Missouri (MO)"), + ("MT", "Montana (MT)"), + ("NE", "Nebraska (NE)"), + ("NV", "Nevada (NV)"), + ("NH", "New Hampshire (NH)"), + ("NJ", "New Jersey (NJ)"), + ("NM", "New Mexico (NM)"), + ("NY", "New York (NY)"), + ("NC", "North Carolina (NC)"), + ("ND", "North Dakota (ND)"), + ("MP", "Northern Mariana Islands (MP)"), + ("OH", "Ohio (OH)"), + ("OK", "Oklahoma (OK)"), + ("OR", "Oregon (OR)"), + ("PA", "Pennsylvania (PA)"), + ("PR", "Puerto Rico (PR)"), + ("RI", "Rhode Island (RI)"), + ("SC", "South Carolina (SC)"), + ("SD", "South Dakota (SD)"), + ("TN", "Tennessee (TN)"), + ("TX", "Texas (TX)"), + ("UM", "United States Minor Outlying Islands (UM)"), + ("UT", "Utah (UT)"), + ("VT", "Vermont (VT)"), + ("VI", "Virgin Islands (VI)"), + ("VA", "Virginia (VA)"), + ("WA", "Washington (WA)"), + ("WV", "West Virginia (WV)"), + ("WI", "Wisconsin (WI)"), + ("WY", "Wyoming (WY)"), + ("AA", "Armed Forces Americas (AA)"), + ("AE", "Armed Forces Africa, Canada, Europe, Middle East (AE)"), + ("AP", "Armed Forces Pacific (AP)"), + ], + max_length=2, + null=True, + verbose_name="state, territory, or military post", + ), + ), + ] diff --git a/src/registrar/models/domain_information.py b/src/registrar/models/domain_information.py index 03b8cc047..d04f09c07 100644 --- a/src/registrar/models/domain_information.py +++ b/src/registrar/models/domain_information.py @@ -159,7 +159,7 @@ class DomainInformation(TimeStampedModel): choices=StateTerritoryChoices.choices, null=True, blank=True, - verbose_name="state / territory", + verbose_name="state, territory, or military post", ) zipcode = models.CharField( max_length=10, diff --git a/src/registrar/models/domain_request.py b/src/registrar/models/domain_request.py index 0cd84fb11..5d1b93be7 100644 --- a/src/registrar/models/domain_request.py +++ b/src/registrar/models/domain_request.py @@ -423,7 +423,7 @@ class DomainRequest(TimeStampedModel): choices=StateTerritoryChoices.choices, null=True, blank=True, - verbose_name="state / territory", + verbose_name="state, territory, or military post", ) zipcode = models.CharField( max_length=10, diff --git a/src/registrar/models/portfolio.py b/src/registrar/models/portfolio.py index fadcf8cac..61d4f7a30 100644 --- a/src/registrar/models/portfolio.py +++ b/src/registrar/models/portfolio.py @@ -89,7 +89,7 @@ class Portfolio(TimeStampedModel): choices=StateTerritoryChoices.choices, null=True, blank=True, - verbose_name="state / territory", + verbose_name="state, territory, or military post", ) zipcode = models.CharField( diff --git a/src/registrar/templates/domain_detail.html b/src/registrar/templates/domain_detail.html index 26f57238a..4b6ca6e77 100644 --- a/src/registrar/templates/domain_detail.html +++ b/src/registrar/templates/domain_detail.html @@ -77,7 +77,7 @@ {% include "includes/summary_item.html" with title='Suborganization' value=domain.domain_info.sub_organization edit_link=url editable=is_editable|and:has_edit_suborganization_portfolio_permission %} {% else %} {% url 'domain-org-name-address' pk=domain.id as url %} - {% include "includes/summary_item.html" with title='Organization name and mailing address' value=domain.domain_info address='true' edit_link=url editable=is_editable %} + {% include "includes/summary_item.html" with title='Organization' value=domain.domain_info address='true' edit_link=url editable=is_editable %} {% url 'domain-senior-official' pk=domain.id as url %} {% include "includes/summary_item.html" with title='Senior official' value=domain.domain_info.senior_official contact='true' edit_link=url editable=is_editable %} diff --git a/src/registrar/templates/domain_org_name_address.html b/src/registrar/templates/domain_org_name_address.html index a7eb02b59..78baed09e 100644 --- a/src/registrar/templates/domain_org_name_address.html +++ b/src/registrar/templates/domain_org_name_address.html @@ -7,7 +7,7 @@ {# this is right after the messages block in the parent template #} {% include "includes/form_errors.html" with form=form %} -

Organization name and mailing address

+

Organization

The name of your organization will be publicly listed as the domain registrant.

diff --git a/src/registrar/templates/domain_sidebar.html b/src/registrar/templates/domain_sidebar.html index b19fa3220..289f544ce 100644 --- a/src/registrar/templates/domain_sidebar.html +++ b/src/registrar/templates/domain_sidebar.html @@ -12,7 +12,7 @@ {% if not portfolio %} {% with url_name="domain-org-name-address" %} - {% include "includes/domain_sidenav_item.html" with item_text="Organization name and mailing address" %} + {% include "includes/domain_sidenav_item.html" with item_text="Organization" %} {% endwith %} {% endif %} diff --git a/src/registrar/templates/emails/includes/domain_request_summary.txt b/src/registrar/templates/emails/includes/domain_request_summary.txt index 9294420c4..c58aef5e1 100644 --- a/src/registrar/templates/emails/includes/domain_request_summary.txt +++ b/src/registrar/templates/emails/includes/domain_request_summary.txt @@ -15,7 +15,7 @@ State-recognized tribe Election office: {{ domain_request.is_election_board|yesno:"Yes,No,Incomplete" }} {% endif %} -Organization name and mailing address: +Organization: {% spaceless %}{{ domain_request.federal_agency }} {{ domain_request.organization_name }} {{ domain_request.address_line1 }}{% if domain_request.address_line2 %} diff --git a/src/registrar/templates/includes/senior_official.html b/src/registrar/templates/includes/senior_official.html index 98afa4dec..0302bc71f 100644 --- a/src/registrar/templates/includes/senior_official.html +++ b/src/registrar/templates/includes/senior_official.html @@ -16,15 +16,7 @@ {% if can_edit %} {% include "includes/required_fields.html" %} -{% else %} -

- The senior official for your organization can’t be updated here. - To suggest an update, email help@get.gov. -

-{% endif %} - -{% if can_edit %} -
+ {% csrf_token %} {% input_with_errors form.first_name %} {% input_with_errors form.last_name %} @@ -33,17 +25,26 @@
{% elif not form.full_name.value and not form.title.value and not form.email.value %} -

No senior official was found.

+

+ Your senior official is a person within your organization who can authorize domain requests. + We don't have information about your organization's senior official. To suggest an update, email help@get.gov. +

{% else %} - {% if form.full_name.value is not None %} - {% include "includes/input_read_only.html" with field=form.full_name %} - {% endif %} +

+ The senior official for your organization can’t be updated here. + To suggest an update, email help@get.gov. +

+
+ {% if form.full_name.value is not None %} + {% include "includes/input_read_only.html" with field=form.full_name %} + {% endif %} - {% if form.title.value is not None %} - {% include "includes/input_read_only.html" with field=form.title %} - {% endif %} + {% if form.title.value is not None %} + {% include "includes/input_read_only.html" with field=form.title %} + {% endif %} - {% if form.email.value is not None %} - {% include "includes/input_read_only.html" with field=form.email %} - {% endif %} + {% if form.email.value is not None %} + {% include "includes/input_read_only.html" with field=form.email %} + {% endif %} +
{% endif %} diff --git a/src/registrar/templates/portfolio_base.html b/src/registrar/templates/portfolio_base.html index f6b97afbc..d39a44879 100644 --- a/src/registrar/templates/portfolio_base.html +++ b/src/registrar/templates/portfolio_base.html @@ -9,9 +9,6 @@ {# the entire logged in page goes here #}
- {% block messages %} - {% include "includes/form_messages.html" %} - {% endblock %} {% block portfolio_content %}{% endblock %} diff --git a/src/registrar/templates/portfolio_domains.html b/src/registrar/templates/portfolio_domains.html index 51011a1a6..dde51ea59 100644 --- a/src/registrar/templates/portfolio_domains.html +++ b/src/registrar/templates/portfolio_domains.html @@ -9,6 +9,10 @@ {% endblock %} {% block portfolio_content %} +{% block messages %} + {% include "includes/form_messages.html" %} +{% endblock %} +

Domains

{% include "includes/domains_table.html" with portfolio=portfolio user_domain_count=user_domain_count %} diff --git a/src/registrar/templates/portfolio_no_domains.html b/src/registrar/templates/portfolio_no_domains.html index e9b0da306..75ff3a91f 100644 --- a/src/registrar/templates/portfolio_no_domains.html +++ b/src/registrar/templates/portfolio_no_domains.html @@ -5,6 +5,12 @@ {% block title %} Domains | {% endblock %} {% block portfolio_content %} + +{% block messages %} + {% include "includes/form_messages.html" %} +{% endblock %} + +

Domains

diff --git a/src/registrar/templates/portfolio_organization.html b/src/registrar/templates/portfolio_organization.html index 4ae035ad4..2e6a1f488 100644 --- a/src/registrar/templates/portfolio_organization.html +++ b/src/registrar/templates/portfolio_organization.html @@ -1,7 +1,7 @@ {% extends 'portfolio_base.html' %} {% load static field_helpers%} -{% block title %}Organization mailing address | {{ portfolio.name }}{% endblock %} +{% block title %}Organization name and mailing address | {{ portfolio.name }}{% endblock %} {% load static %} @@ -19,21 +19,25 @@
+ {% block messages %} + {% include "includes/form_messages.html" %} + {% endblock %} +

Organization

-

The name of your federal agency will be publicly listed as the domain registrant.

+

The name of your organization will be publicly listed as the domain registrant.

{% if has_edit_org_portfolio_permission %}

- The federal agency for your organization can’t be updated here. + Your organization name can’t be updated here. To suggest an update, email help@get.gov.

{% include "includes/form_errors.html" with form=form %} {% include "includes/required_fields.html" %} -
+ {% csrf_token %} -

Federal agency

+

Organization name

{{ portfolio.federal_agency }}

@@ -49,7 +53,7 @@
{% else %} -

Federal agency

+

Organization name

{{ portfolio.federal_agency }}

diff --git a/src/registrar/templates/portfolio_requests.html b/src/registrar/templates/portfolio_requests.html index 70d17feae..d21bbcc4e 100644 --- a/src/registrar/templates/portfolio_requests.html +++ b/src/registrar/templates/portfolio_requests.html @@ -9,6 +9,10 @@ {% endblock %} {% block portfolio_content %} +{% block messages %} + {% include "includes/form_messages.html" %} +{% endblock %} +

Domain requests

diff --git a/src/registrar/templates/portfolio_senior_official.html b/src/registrar/templates/portfolio_senior_official.html index 51ef83be5..631db0206 100644 --- a/src/registrar/templates/portfolio_senior_official.html +++ b/src/registrar/templates/portfolio_senior_official.html @@ -6,6 +6,10 @@ {% load static %} {% block portfolio_content %} +{% block messages %} + {% include "includes/form_messages.html" %} +{% endblock %} +

") @less_console_noise_decorator def test_domain_org_name_address_content(self): @@ -1615,7 +1617,7 @@ class TestDomainSuborganization(TestDomainOverview): # Test for the title change self.assertContains(page, "Suborganization") - self.assertNotContains(page, "Organization name") + self.assertNotContains(page, "Organization") # Test for the good value self.assertContains(page, "Ice Cream") diff --git a/src/registrar/tests/test_views_portfolio.py b/src/registrar/tests/test_views_portfolio.py index b8392a370..abb8f8178 100644 --- a/src/registrar/tests/test_views_portfolio.py +++ b/src/registrar/tests/test_views_portfolio.py @@ -200,7 +200,7 @@ class TestPortfolio(WebTest): # Assert the response is a 200 self.assertEqual(response.status_code, 200) # The label for Federal agency will always be a h4 - self.assertContains(response, '

Federal agency

') + self.assertContains(response, '

Organization name

') # The read only label for city will be a h4 self.assertContains(response, '

City

') self.assertNotContains(response, 'for="id_city"') @@ -225,10 +225,10 @@ class TestPortfolio(WebTest): # Assert the response is a 200 self.assertEqual(response.status_code, 200) # The label for Federal agency will always be a h4 - self.assertContains(response, '

Federal agency

') + self.assertContains(response, '

Organization name

') # The read only label for city will be a h4 self.assertNotContains(response, '

City

') - self.assertNotContains(response, '

Los Angeles

>') + self.assertNotContains(response, '

Los Angeles

') self.assertContains(response, 'for="id_city"') @less_console_noise_decorator @@ -342,9 +342,7 @@ class TestPortfolio(WebTest): user=self.user, portfolio=self.portfolio, additional_permissions=portfolio_additional_permissions ) page = self.app.get(reverse("organization")) - self.assertContains( - page, "The name of your federal agency will be publicly listed as the domain registrant." - ) + self.assertContains(page, "The name of your organization will be publicly listed as the domain registrant.") @less_console_noise_decorator def test_domain_org_name_address_content(self): diff --git a/src/registrar/views/domain.py b/src/registrar/views/domain.py index 174f01ecc..db0572bb3 100644 --- a/src/registrar/views/domain.py +++ b/src/registrar/views/domain.py @@ -204,7 +204,7 @@ class DomainView(DomainBaseView): class DomainOrgNameAddressView(DomainFormBaseView): - """Organization name and mailing address view""" + """Organization view""" model = Domain template_name = "domain_org_name_address.html" diff --git a/src/registrar/views/domain_request.py b/src/registrar/views/domain_request.py index 06c0a7fc7..4fe59c70c 100644 --- a/src/registrar/views/domain_request.py +++ b/src/registrar/views/domain_request.py @@ -83,7 +83,7 @@ class DomainRequestWizard(DomainRequestWizardPermissionView, TemplateView): Step.TRIBAL_GOVERNMENT: _("Tribal government"), Step.ORGANIZATION_FEDERAL: _("Federal government branch"), Step.ORGANIZATION_ELECTION: _("Election office"), - Step.ORGANIZATION_CONTACT: _("Organization name and mailing address"), + Step.ORGANIZATION_CONTACT: _("Organization"), Step.ABOUT_YOUR_ORGANIZATION: _("About your organization"), Step.SENIOR_OFFICIAL: _("Senior official"), Step.CURRENT_SITES: _("Current websites"),