diff --git a/src/registrar/admin.py b/src/registrar/admin.py index f187dfdb1..82daee6f8 100644 --- a/src/registrar/admin.py +++ b/src/registrar/admin.py @@ -484,8 +484,7 @@ class DomainApplicationAdmin(ListHeaderAdmin): "federal_agency", "federal_type", "is_election_board", - "type_of_work", - "more_organization_information", + "about_your_organization", ] }, ), @@ -523,8 +522,7 @@ class DomainApplicationAdmin(ListHeaderAdmin): # Read only that we'll leverage for CISA Analysts analyst_readonly_fields = [ "creator", - "type_of_work", - "more_organization_information", + "about_your_organization", "address_line1", "address_line2", "zipcode", diff --git a/src/registrar/config/urls.py b/src/registrar/config/urls.py index 0f136c932..9c3624c2c 100644 --- a/src/registrar/config/urls.py +++ b/src/registrar/config/urls.py @@ -27,7 +27,7 @@ for step, view in [ (Step.ORGANIZATION_FEDERAL, views.OrganizationFederal), (Step.ORGANIZATION_ELECTION, views.OrganizationElection), (Step.ORGANIZATION_CONTACT, views.OrganizationContact), - (Step.TYPE_OF_WORK, views.TypeOfWork), + (Step.ABOUT_YOUR_ORGANIZATION, views.AboutYourOrganization), (Step.AUTHORIZING_OFFICIAL, views.AuthorizingOfficial), (Step.CURRENT_SITES, views.CurrentSites), (Step.DOTGOV_DOMAIN, views.DotgovDomain), diff --git a/src/registrar/forms/application_wizard.py b/src/registrar/forms/application_wizard.py index 578a501d3..93ec18aad 100644 --- a/src/registrar/forms/application_wizard.py +++ b/src/registrar/forms/application_wizard.py @@ -310,28 +310,9 @@ class OrganizationContactForm(RegistrarForm): return federal_agency -class TypeOfWorkForm(RegistrarForm): - type_of_work = forms.CharField( - # label has to end in a space to get the label_suffix to show - label="What type of work does your organization do? ", - widget=forms.Textarea(), - validators=[ - MaxLengthValidator( - 1000, - message="Response must be less than 1000 characters.", - ) - ], - error_messages={"required": "Enter the type of work your organization does."}, - ) - - more_organization_information = forms.CharField( - # label has to end in a space to get the label_suffix to show - label=( - "Describe how your organization is a government organization that is" - " independent of a state government. Include links to authorizing" - " legislation, applicable bylaws or charter, or other documentation to" - " support your claims. " - ), +class AboutYourOrganizationForm(RegistrarForm): + about_your_organization = forms.CharField( + label="About your organization", widget=forms.Textarea(), validators=[ MaxLengthValidator( @@ -340,9 +321,7 @@ class TypeOfWorkForm(RegistrarForm): ) ], error_messages={ - "required": ( - "Describe how your organization is independent of a state government." - ) + "required": ("Enter more information about your organization.") }, ) diff --git a/src/registrar/migrations/0031_alter_domain_state.py b/src/registrar/migrations/0031_alter_domain_state.py deleted file mode 100644 index 2545adb27..000000000 --- a/src/registrar/migrations/0031_alter_domain_state.py +++ /dev/null @@ -1,30 +0,0 @@ -# Generated by Django 4.2.1 on 2023-09-07 17:53 - -from django.db import migrations -import django_fsm - - -class Migration(migrations.Migration): - dependencies = [ - ("registrar", "0030_alter_user_status"), - ] - - operations = [ - migrations.AlterField( - model_name="domain", - name="state", - field=django_fsm.FSMField( - choices=[ - ("created", "Created"), - ("deleted", "Deleted"), - ("unknown", "Unknown"), - ("ready", "Ready"), - ("onhold", "Onhold"), - ], - default="unknown", - help_text="Very basic info about the lifecycle of this domain object", - max_length=21, - protected=True, - ), - ), - ] diff --git a/src/registrar/migrations/0031_transitiondomain.py b/src/registrar/migrations/0031_transitiondomain.py deleted file mode 100644 index e72a8d85a..000000000 --- a/src/registrar/migrations/0031_transitiondomain.py +++ /dev/null @@ -1,60 +0,0 @@ -# Generated by Django 4.2.1 on 2023-09-11 14:44 - -from django.db import migrations, models - - -class Migration(migrations.Migration): - dependencies = [ - ("registrar", "0030_alter_user_status"), - ] - - operations = [ - migrations.CreateModel( - name="TransitionDomain", - fields=[ - ( - "id", - models.BigAutoField( - auto_created=True, - primary_key=True, - serialize=False, - verbose_name="ID", - ), - ), - ("created_at", models.DateTimeField(auto_now_add=True)), - ("updated_at", models.DateTimeField(auto_now=True)), - ( - "username", - models.TextField( - help_text="Username - this will be an email address", - verbose_name="Username", - ), - ), - ( - "domain_name", - models.TextField(blank=True, null=True, verbose_name="Domain name"), - ), - ( - "status", - models.CharField( - blank=True, - choices=[("created", "Created"), ("hold", "Hold")], - help_text="domain status during the transfer", - max_length=255, - verbose_name="Status", - ), - ), - ( - "email_sent", - models.BooleanField( - default=False, - help_text="indicates whether email was sent", - verbose_name="email sent", - ), - ), - ], - options={ - "abstract": False, - }, - ), - ] diff --git a/src/registrar/migrations/0031_transitiondomain_and_more.py b/src/registrar/migrations/0031_transitiondomain_and_more.py new file mode 100644 index 000000000..79bf7eab4 --- /dev/null +++ b/src/registrar/migrations/0031_transitiondomain_and_more.py @@ -0,0 +1,122 @@ +# Generated by Django 4.2.1 on 2023-09-13 22:25 + +from django.db import migrations, models +import django_fsm + + +class Migration(migrations.Migration): + dependencies = [ + ("registrar", "0030_alter_user_status"), + ] + + operations = [ + migrations.CreateModel( + name="TransitionDomain", + fields=[ + ( + "id", + models.BigAutoField( + auto_created=True, + primary_key=True, + serialize=False, + verbose_name="ID", + ), + ), + ("created_at", models.DateTimeField(auto_now_add=True)), + ("updated_at", models.DateTimeField(auto_now=True)), + ( + "username", + models.TextField( + help_text="Username - this will be an email address", + verbose_name="Username", + ), + ), + ( + "domain_name", + models.TextField(blank=True, null=True, verbose_name="Domain name"), + ), + ( + "status", + models.CharField( + blank=True, + choices=[("created", "Created"), ("hold", "Hold")], + help_text="domain status during the transfer", + max_length=255, + verbose_name="Status", + ), + ), + ( + "email_sent", + models.BooleanField( + default=False, + help_text="indicates whether email was sent", + verbose_name="email sent", + ), + ), + ], + options={ + "abstract": False, + }, + ), + migrations.RemoveField( + model_name="domainapplication", + name="more_organization_information", + ), + migrations.RemoveField( + model_name="domainapplication", + name="type_of_work", + ), + migrations.RemoveField( + model_name="domaininformation", + name="more_organization_information", + ), + migrations.RemoveField( + model_name="domaininformation", + name="type_of_work", + ), + migrations.AddField( + model_name="domainapplication", + name="about_your_organization", + field=models.TextField( + blank=True, help_text="Information about your organization", null=True + ), + ), + migrations.AddField( + model_name="domaininformation", + name="about_your_organization", + field=models.TextField( + blank=True, help_text="Information about your organization", null=True + ), + ), + migrations.AlterField( + model_name="domain", + name="state", + field=django_fsm.FSMField( + choices=[ + ("unknown", "Unknown"), + ("dns needed", "Dns Needed"), + ("ready", "Ready"), + ("on hold", "On Hold"), + ("deleted", "Deleted"), + ], + default="unknown", + help_text="Very basic info about the lifecycle of this domain object", + max_length=21, + protected=True, + ), + ), + migrations.AlterField( + model_name="publiccontact", + name="contact_type", + field=models.CharField( + choices=[ + ("registrant", "Registrant"), + ("admin", "Administrative"), + ("tech", "Technical"), + ("security", "Security"), + ], + help_text="For which type of WHOIS contact", + max_length=14, + ), + ), + ] diff --git a/src/registrar/migrations/0032_merge_0031_alter_domain_state_0031_transitiondomain.py b/src/registrar/migrations/0032_merge_0031_alter_domain_state_0031_transitiondomain.py deleted file mode 100644 index 4c0a38427..000000000 --- a/src/registrar/migrations/0032_merge_0031_alter_domain_state_0031_transitiondomain.py +++ /dev/null @@ -1,12 +0,0 @@ -# Generated by Django 4.2.1 on 2023-09-12 14:12 - -from django.db import migrations - - -class Migration(migrations.Migration): - dependencies = [ - ("registrar", "0031_alter_domain_state"), - ("registrar", "0031_transitiondomain"), - ] - - operations = [] diff --git a/src/registrar/migrations/0033_alter_domain_state_alter_publiccontact_contact_type.py b/src/registrar/migrations/0033_alter_domain_state_alter_publiccontact_contact_type.py deleted file mode 100644 index 57f05de14..000000000 --- a/src/registrar/migrations/0033_alter_domain_state_alter_publiccontact_contact_type.py +++ /dev/null @@ -1,44 +0,0 @@ -# Generated by Django 4.2.1 on 2023-09-13 00:46 - -from django.db import migrations, models -import django_fsm - - -class Migration(migrations.Migration): - dependencies = [ - ("registrar", "0032_merge_0031_alter_domain_state_0031_transitiondomain"), - ] - - operations = [ - migrations.AlterField( - model_name="domain", - name="state", - field=django_fsm.FSMField( - choices=[ - ("unknown", "Unknown"), - ("dns needed", "Dns Needed"), - ("ready", "Ready"), - ("on hold", "On Hold"), - ("deleted", "Deleted"), - ], - default="unknown", - help_text="Very basic info about the lifecycle of this domain object", - max_length=21, - protected=True, - ), - ), - migrations.AlterField( - model_name="publiccontact", - name="contact_type", - field=models.CharField( - choices=[ - ("registrant", "Registrant"), - ("admin", "Administrative"), - ("tech", "Technical"), - ("security", "Security"), - ], - help_text="For which type of WHOIS contact", - max_length=14, - ), - ), - ] diff --git a/src/registrar/models/domain_application.py b/src/registrar/models/domain_application.py index b1230b703..c78a510dc 100644 --- a/src/registrar/models/domain_application.py +++ b/src/registrar/models/domain_application.py @@ -378,16 +378,10 @@ class DomainApplication(TimeStampedModel): help_text="Urbanization (Puerto Rico only)", ) - type_of_work = models.TextField( + about_your_organization = models.TextField( null=True, blank=True, - help_text="Type of work of the organization", - ) - - more_organization_information = models.TextField( - null=True, - blank=True, - help_text="More information about your organization", + help_text="Information about your organization", ) authorizing_official = models.ForeignKey( @@ -653,7 +647,7 @@ class DomainApplication(TimeStampedModel): ] return bool(user_choice and user_choice not in excluded) - def show_type_of_work(self) -> bool: + def show_about_your_organization(self) -> bool: """Show this step if this is a special district or interstate.""" user_choice = self.organization_type return user_choice in [ diff --git a/src/registrar/models/domain_information.py b/src/registrar/models/domain_information.py index b12039e73..c1c6142d0 100644 --- a/src/registrar/models/domain_information.py +++ b/src/registrar/models/domain_information.py @@ -134,16 +134,10 @@ class DomainInformation(TimeStampedModel): verbose_name="Urbanization (Puerto Rico only)", ) - type_of_work = models.TextField( + about_your_organization = models.TextField( null=True, blank=True, - help_text="Type of work of the organization", - ) - - more_organization_information = models.TextField( - null=True, - blank=True, - help_text="Further information about the government organization", + help_text="Information about your organization", ) authorizing_official = models.ForeignKey( diff --git a/src/registrar/templates/application_about_your_organization.html b/src/registrar/templates/application_about_your_organization.html new file mode 100644 index 000000000..f1b843b7a --- /dev/null +++ b/src/registrar/templates/application_about_your_organization.html @@ -0,0 +1,23 @@ +{% extends 'application_form.html' %} +{% load field_helpers %} + +{% block form_instructions %} +
We’d like to know more about your organization. Include the following in your response:
+ +*This question is required.
+{% endblock %} + +{% block form_fields %} + {% with attr_maxlength=1000 add_label_class="usa-sr-only" %} + {% input_with_errors forms.0.about_your_organization %} + {% endwith %} +{% endblock %} \ No newline at end of file diff --git a/src/registrar/templates/application_review.html b/src/registrar/templates/application_review.html index b9ac97871..be81303b8 100644 --- a/src/registrar/templates/application_review.html +++ b/src/registrar/templates/application_review.html @@ -46,9 +46,8 @@ Incomplete {% endif %} {% endif %} - {% if step == Step.TYPE_OF_WORK %} -{{ application.type_of_work|default:"Incomplete" }}
-{{ application.more_organization_information|default:"Incomplete" }}
+ {% if step == Step.ABOUT_YOUR_ORGANIZATION %} +{{ application.about_your_organization|default:"Incomplete" }}
{% endif %} {% if step == Step.AUTHORIZING_OFFICIAL %} {% if application.authorizing_official %} diff --git a/src/registrar/templates/application_status.html b/src/registrar/templates/application_status.html index 67e8e7664..a68c07c8a 100644 --- a/src/registrar/templates/application_status.html +++ b/src/registrar/templates/application_status.html @@ -77,12 +77,8 @@ {% include "includes/summary_item.html" with title='Organization name and mailing address' value=domainapplication address='true' heading_level=heading_level %} {% endif %} - {% if domainapplication.type_of_work %} - {% include "includes/summary_item.html" with title='Type of work' value=domainapplication.type_of_work heading_level=heading_level %} - {% endif %} - - {% if domainapplication.more_organization_information %} - {% include "includes/summary_item.html" with title='More information about your organization' value=domainapplication.more_organization_information heading_level=heading_level %} + {% if domainapplication.about_your_organization %} + {% include "includes/summary_item.html" with title='About your organization' value=domainapplication.about_your_organization heading_level=heading_level %} {% endif %} {% if domainapplication.authorizing_official %} diff --git a/src/registrar/templates/application_type_of_work.html b/src/registrar/templates/application_type_of_work.html deleted file mode 100644 index 9ad58936f..000000000 --- a/src/registrar/templates/application_type_of_work.html +++ /dev/null @@ -1,10 +0,0 @@ -{% extends 'application_form.html' %} -{% load field_helpers %} - - -{% block form_fields %} - {% with attr_maxlength=1000 %} - {% input_with_errors forms.0.type_of_work %} - {% input_with_errors forms.0.more_organization_information %} - {% endwith %} -{% endblock %} \ No newline at end of file diff --git a/src/registrar/templates/domain_users.html b/src/registrar/templates/domain_users.html index 09d391dc8..22b9d18d1 100644 --- a/src/registrar/templates/domain_users.html +++ b/src/registrar/templates/domain_users.html @@ -20,7 +20,7 @@ {% for permission in domain.permissions.all %}