require comments and add "non veteran" option in mentor request form

Author: chynh

pybot/endpoints/slack/message_templates/commands.py

@@ -78,7 +78,7 @@ def mentor_request_blocks(services, skillsets):
         {
             "type": "section",
             "block_id": "comments",
-            "text": {"type": "mrkdwn", "text": "*Add additional comments* (optional)"},
+            "text": {"type": "mrkdwn", "text": "*Add comments* (required)"},
             "accessory": {
                 "type": "button",
                 "action_id": "comments_btn",
@@ -111,6 +111,10 @@ def mentor_request_blocks(services, skillsets):
                         "text": {"type": "plain_text", "text": "Military Spouse"},
                         "value": "Military Spouse",
                     },
+                    {
+                        "text": {"type": "plain_text", "text": "Non Veteran"},
+                        "value": "Non Veteran",
+                    },
                 ],
             },
         },

pybot/endpoints/slack/message_templates/mentor_request.py

@@ -81,14 +81,14 @@ def affiliation(self, new_affiliation: str) -> None:
             self.clear_errors()
 
     def validate_self(self) -> bool:
-        if not self.service or not self.affiliation:
+        if not self.service or not self.affiliation or not self.details:
             return False
         self.clear_errors()
         return True
 
     def add_errors(self) -> None:
         submit_attachment = {
-            "text": ":warning: Service and group certification are required. :warning:",
+            "text": ":warning: Service, group certification and comments are required. :warning:",
             "color": "danger",
         }
         self.attachments = [submit_attachment]