diff --git a/src/registrar/config/urls.py b/src/registrar/config/urls.py index 53b83e564..66a8a9b74 100644 --- a/src/registrar/config/urls.py +++ b/src/registrar/config/urls.py @@ -46,8 +46,8 @@ DOMAIN_REQUEST_NAMESPACE = views.DomainRequestWizard.URL_NAMESPACE # dynamically generate the other domain_request_urls domain_request_urls = [ path("", RedirectView.as_view(pattern_name="domain-request:start"), name="redirect-to-start"), - path("start/", views.DomainRequestWizard.as_view(), name="start"), - path("finished/", views.Finished.as_view(), name="finished"), + path("start/", views.DomainRequestWizard.as_view(), name=views.DomainRequestWizard.NEW_URL_NAME), + path("finished/", views.Finished.as_view(), name=views.DomainRequestWizard.FINISHED_URL_NAME), ] for step, view in [ # add/remove steps here diff --git a/src/registrar/forms/domain_request_wizard.py b/src/registrar/forms/domain_request_wizard.py index e55c40858..95d6571d6 100644 --- a/src/registrar/forms/domain_request_wizard.py +++ b/src/registrar/forms/domain_request_wizard.py @@ -794,6 +794,22 @@ class AnythingElseForm(BaseDeletableRegistrarForm): ) +class PortfolioAnythingElseForm(BaseDeletableRegistrarForm): + """The form for the portfolio additional details page. Tied to the anything_else field.""" + + anything_else = forms.CharField( + required=False, + label="Anything else?", + widget=forms.Textarea(), + validators=[ + MaxLengthValidator( + 2000, + message="Response must be less than 2000 characters.", + ) + ], + ) + + class AnythingElseYesNoForm(BaseYesNoForm): """Yes/no toggle for the anything else question on additional details""" diff --git a/src/registrar/templates/includes/portfolio_request_review_steps.html b/src/registrar/templates/includes/portfolio_request_review_steps.html index 5c6e64269..fcb087090 100644 --- a/src/registrar/templates/includes/portfolio_request_review_steps.html +++ b/src/registrar/templates/includes/portfolio_request_review_steps.html @@ -62,7 +62,7 @@ {% endif %} {% if step == Step.ADDITIONAL_DETAILS %} - {% with title=form_titles|get_item:step value=domain_request.anything_else|default:"Incomplete"|safe %} + {% with title=form_titles|get_item:step value=domain_request.anything_else|default:"None" %} {% include "includes/summary_item.html" with title=title value=value heading_level=heading_level editable=is_editable edit_link=domain_request_url %} {% endwith %} {% endif %} diff --git a/src/registrar/templates/portfolio_domain_request_additional_details.html b/src/registrar/templates/portfolio_domain_request_additional_details.html index 3c5b50d6b..5bc529243 100644 --- a/src/registrar/templates/portfolio_domain_request_additional_details.html +++ b/src/registrar/templates/portfolio_domain_request_additional_details.html @@ -2,18 +2,18 @@ {% load static field_helpers %} {% block form_required_fields_help_text %} -{% include "includes/required_fields.html" %} +{% comment %} Empty - this step is not required {% endcomment %} {% endblock %} {% block form_fields %} -
-

Is there anything else you’d like us to know about your domain request?

+
+

Is there anything else you’d like us to know about your domain request?

-
-

Provide details below. *

+
+

This question is optional.

{% with attr_maxlength=2000 add_label_class="usa-sr-only" %} {% input_with_errors forms.0.anything_else %} {% endwith %} diff --git a/src/registrar/views/domain_request.py b/src/registrar/views/domain_request.py index da194755f..85f7576d0 100644 --- a/src/registrar/views/domain_request.py +++ b/src/registrar/views/domain_request.py @@ -53,7 +53,8 @@ class DomainRequestWizard(DomainRequestWizardPermissionView, TemplateView): URL_NAMESPACE = "domain-request" # name for accessing /domain-request//edit EDIT_URL_NAME = "edit-domain-request" - NEW_URL_NAME = "/request/start/" + NEW_URL_NAME = "start" + FINISHED_URL_NAME = "finished" # region: Titles # We need to pass our human-readable step titles as context to the templates. @@ -313,7 +314,7 @@ class DomainRequestWizard(DomainRequestWizardPermissionView, TemplateView): # send users "to the domain request wizard" without needing to know which view # is first in the list of steps. if self.__class__ == DomainRequestWizard: - if request.path_info == self.NEW_URL_NAME: + if current_url == self.NEW_URL_NAME: # Clear context so the prop getter won't create a request here. # Creating a request will be handled in the post method for the # intro page. @@ -614,7 +615,7 @@ class RequestingEntity(DomainRequestWizard): class PortfolioAdditionalDetails(DomainRequestWizard): template_name = "portfolio_domain_request_additional_details.html" - forms = [forms.AnythingElseForm] + forms = [forms.PortfolioAnythingElseForm] # Non-portfolio pages