diff --git a/src/registrar/forms/application_wizard.py b/src/registrar/forms/application_wizard.py index ff40ad3e4..56d62ce2b 100644 --- a/src/registrar/forms/application_wizard.py +++ b/src/registrar/forms/application_wizard.py @@ -91,7 +91,10 @@ class OrganizationContactForm(RegistrarForm): choices=[("", "--Select--")] + DomainApplication.StateTerritoryChoices.choices, ) zipcode = forms.CharField(label="ZIP code") - urbanization = forms.CharField(label="Urbanization (Puerto Rico only)") + urbanization = forms.CharField( + required=False, + label="Urbanization (Puerto Rico only)", + ) class AuthorizingOfficialForm(RegistrarForm): diff --git a/src/registrar/migrations/0005_domainapplication_city_and_more.py b/src/registrar/migrations/0005_domainapplication_city_and_more.py index 367794e2a..ea5d6b57f 100644 --- a/src/registrar/migrations/0005_domainapplication_city_and_more.py +++ b/src/registrar/migrations/0005_domainapplication_city_and_more.py @@ -1,4 +1,4 @@ -# Generated by Django 4.1.3 on 2022-12-09 19:27 +# Generated by Django 4.1.3 on 2022-12-12 21:00 from django.db import migrations, models @@ -20,4 +20,11 @@ class Migration(migrations.Migration): name="urbanization", field=models.TextField(blank=True, help_text="Urbanization", null=True), ), + migrations.AlterField( + model_name="domainapplication", + name="federal_agency", + field=models.TextField( + blank=True, help_text="Top level federal agency", null=True + ), + ), ] diff --git a/src/registrar/migrations/0006_alter_domainapplication_federal_agency.py b/src/registrar/migrations/0006_alter_domainapplication_federal_agency.py deleted file mode 100644 index a214b4666..000000000 --- a/src/registrar/migrations/0006_alter_domainapplication_federal_agency.py +++ /dev/null @@ -1,20 +0,0 @@ -# Generated by Django 4.1.3 on 2022-12-12 14:48 - -from django.db import migrations, models - - -class Migration(migrations.Migration): - - dependencies = [ - ("registrar", "0005_domainapplication_city_and_more"), - ] - - operations = [ - migrations.AlterField( - model_name="domainapplication", - name="federal_agency", - field=models.TextField( - blank=True, help_text="Top level federal agency", null=True - ), - ), - ] diff --git a/src/registrar/templates/application_review.html b/src/registrar/templates/application_review.html index 1be19cade..33ac78921 100644 --- a/src/registrar/templates/application_review.html +++ b/src/registrar/templates/application_review.html @@ -9,25 +9,34 @@ {% csrf_token %} {% for step in wizard.steps.all|slice:":-1" %} -
+
+
{{ form_titles|get_item:step }}
{% if step == step_cls.ORGANIZATION_TYPE %} - {{ application.get_organization_type_display }} + {{ application.get_organization_type_display|default:"Incomplete" }} {% endif %} {% if step == step_cls.ORGANIZATION_FEDERAL %} - {{ application.get_federal_type_display }} + {{ application.get_federal_type_display|default:"Incomplete" }} {% endif %} {% if step == step_cls.ORGANIZATION_ELECTION %} - {{ application.is_election_board|yesno:"Yes,No,Maybe" }} + {{ application.is_election_board|yesno:"Yes,No,Incomplete" }} {% endif %} {% if step == step_cls.ORGANIZATION_CONTACT %} + {% if organization.organization_name %} {% include "includes/organization_address.html" with organization=application %} + {% else %} + Incomplete + {% endif %} {% endif %} {% if step == step_cls.AUTHORIZING_OFFICIAL %} + {% if application.authorizing_official %} {% include "includes/contact.html" with contact=application.authorizing_official %} + {% else %} + Incomplete + {% endif %} {% endif %} {% if step == step_cls.CURRENT_SITES %}
    @@ -40,17 +49,21 @@ {% endif %} {% if step == step_cls.DOTGOV_DOMAIN %}
      -
    • {{ application.requested_domain.name }}
    • +
    • {{ application.requested_domain.name|default:"Incomplete" }}
    • {% for site in application.alternative_domains.all %}
    • {{ site.website }}
    • {% endfor %}
    {% endif %} {% if step == step_cls.PURPOSE %} - {{ application.purpose }} + {{ application.purpose|default:"Incomplete" }} {% endif %} {% if step == step_cls.YOUR_CONTACT %} - {% include "includes/contact.html" with contact=application.submitter %} + {% if application.submitter %} + {% include "includes/contact.html" with contact=application.submitter %} + {% else %} + Incomplete + {% endif %} {% endif %} {% if step == step_cls.OTHER_CONTACTS %} {% for other in application.other_contacts.all %} @@ -60,10 +73,13 @@ {% endfor %} {% endif %} {% if step == step_cls.SECURITY_EMAIL %} - {{ application.security_email }} + {{ application.security_email|default:"None" }} {% endif %} {% if step == step_cls.ANYTHING_ELSE %} - {{ application.anything_else }} + {{ application.anything_else|default:"No" }} + {% endif %} + {% if step == step_cls.REQUIREMENTS %} + {{ application.is_policy_acknowledged|yesno:"Agree,Do not agree,Do not agree" }} {% endif %}
@@ -72,7 +88,7 @@ href="{% url wizard.url_name step=step %}" >Edit {{ form_titles|get_item:step }}
-
+ {% endfor %} {{ block.super }}