From 369d4ccb5186e07876de84cd749cde1b35a76a21 Mon Sep 17 00:00:00 2001
From: Charles Shin <charles@openroad.ca>
Date: Tue, 4 Dec 2018 16:41:19 -0800
Subject: [PATCH] DIV-905: Update postal code to be not required field.

---
 .../apps/core/templates/question/10_other_questions.html    | 4 ++--
 edivorce/apps/core/tests.py                                 | 2 +-
 edivorce/fixtures/Question.json                             | 6 ++----
 3 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/edivorce/apps/core/templates/question/10_other_questions.html b/edivorce/apps/core/templates/question/10_other_questions.html
index 5b9a0c6d..36a2922d 100644
--- a/edivorce/apps/core/templates/question/10_other_questions.html
+++ b/edivorce/apps/core/templates/question/10_other_questions.html
@@ -28,7 +28,7 @@
         <div class="radio"><label>{% input_field type="radio" class="radio-with-other" name="address_to_send_official_document_country_you" value="USA" %}USA</label></div>
         <div class="radio"><label>{% input_field type="radio" class="radio-with-other radio_with_textbox" name="address_to_send_official_document_country_you" value="Other" %}Other {% input_field type="text" name="address_to_send_official_document_other_country_you" class="response-textbox other-textbox input-inline" tabindex="-1" %}</label></div>
 
-        <p>Postal code</p>
+        <p>Postal code (optional)</p>
         {% input_field type="text" name="address_to_send_official_document_postal_code_you" class="form-block input-wide response-textbox" %}
     </div>
 
@@ -62,7 +62,7 @@
         <div class="radio"><label>{% input_field type="radio" class="radio-with-other" name="address_to_send_official_document_country_spouse" value="USA" %}USA</label></div>
         <div class="radio"><label>{% input_field type="radio" class="radio-with-other radio_with_textbox" name="address_to_send_official_document_country_spouse" value="Other" %}Other {% input_field type="text" name="address_to_send_official_document_other_country_spouse" class="response-textbox other-textbox input-inline" tabindex="-1" %}</label></div>
 
-        <p>Postal code</p>
+        <p>Postal code (optional)</p>
         {% input_field type="text" name="address_to_send_official_document_postal_code_spouse" class="form-block input-wide response-textbox" %}
     </div>
 
diff --git a/edivorce/apps/core/tests.py b/edivorce/apps/core/tests.py
index a49ef72a..9c905869 100644
--- a/edivorce/apps/core/tests.py
+++ b/edivorce/apps/core/tests.py
@@ -511,7 +511,7 @@ class UserResponseTestCase(TestCase):
                                                                             'question__conditional_target',
                                                                             'question__reveal_response',
                                                                             'question__required')
-        self.assertEqual(is_complete(step, lst)[0], False)
+        self.assertEqual(is_complete(step, lst)[0], True)
 
         # All required questions for spouse and you
         create_response(user, 'address_to_send_official_document_postal_code_spouse', 'Canada')
diff --git a/edivorce/fixtures/Question.json b/edivorce/fixtures/Question.json
index 18c47a10..f9d0a105 100644
--- a/edivorce/fixtures/Question.json
+++ b/edivorce/fixtures/Question.json
@@ -591,8 +591,7 @@
     "fields": {
         "name": "Postal code",
         "description": "For step 9, Form 1 8. Claimants' addresses for service, Form 38(joint) Affidavit section",
-        "summary_order": 57,
-        "required": "Required"
+        "summary_order": 57
     },
     "model": "core.question",
     "pk": "address_to_send_official_document_postal_code_you"
@@ -671,8 +670,7 @@
     "fields": {
         "name": "Postal code",
         "description": "For step 9, Form 1 8. Claimants' addresses for service",
-        "summary_order": 65,
-        "required": "Required"
+        "summary_order": 65
     },
     "model": "core.question",
     "pk": "address_to_send_official_document_postal_code_spouse"