Allow volunteers to select more than 10 mentor skillsets

Author: AllenAnthes

pybot/endpoints/slack/actions/mentor_volunteer.py

@@ -74,7 +74,7 @@ async def build_airtable_fields(action, request, user_info):
     airtable_fields = {
         "Slack Name": username,
         "Full Name": name,
-        "Skillsets": request.skillsets,
+        "Skillsets": request.skillsets[1:],  # hack to filter out empty first option
         "Email": email,
     }
     return airtable_fields

pybot/endpoints/slack/message_templates/commands.py

@@ -60,7 +60,7 @@ def mentor_request_blocks(services, mentors, skillsets):
                         "text": {"type": "plain_text", "text": skillset},
                         "value": skillset,
                     }
-                    for skillset in skillsets
+                    for skillset in sorted(skillsets)
                 ],
             },
         },
@@ -181,14 +181,15 @@ def mentor_volunteer_blocks(skillsets: List[str]) -> List[dict]:
                         "text": {"type": "plain_text", "text": skillset},
                         "value": skillset,
                     }
-                    for skillset in skillsets
+                    for skillset in sorted(skillsets)
                 ],
             },
         },
         {
             "type": "section",
             "block_id": "clear_volunteer_skillsets",
             "text": {"type": "mrkdwn", "text": "*Selected Skillsets*"},
+            "fields": [{"type": "plain_text", "text": " ", "emoji": True}],
             "accessory": {
                 "type": "button",
                 "action_id": "clear_skillsets_btn",

pybot/endpoints/slack/message_templates/mentor_volunteer.py

@@ -21,27 +21,26 @@ def __init__(self, raw_action: MutableMapping):
 
     @property
     def skillsets(self) -> [str]:
-        if self.skillset_fields:
-            return [field["text"] for field in self.skillset_fields]
-        return []
+        skillset_field = self.skillset_field_text
+        return skillset_field.split("\n")
 
     @property
-    def skillset_fields(self) -> list:
-        return self.blocks[VolunteerBlockIndex.SELECTED_SKILLSETS].get("fields", [])
+    def skillset_field_text(self) -> str:
+        return self.blocks[VolunteerBlockIndex.SELECTED_SKILLSETS]["fields"][0]["text"]
+
+    @skillset_field_text.setter
+    def skillset_field_text(self, value):
+        self.blocks[VolunteerBlockIndex.SELECTED_SKILLSETS]["fields"][0]["text"] = value
 
     def add_skillset(self, skillset: str) -> None:
         """
         Appends the new skillset to the displayed skillsets
         """
         if skillset not in self.skillsets:
-            new_field = {"type": "plain_text", "text": skillset, "emoji": True}
-            self.blocks[VolunteerBlockIndex.SELECTED_SKILLSETS].setdefault(
-                "fields", []
-            ).append(new_field)
+            self.skillset_field_text += f"\n{skillset}"
 
     def clear_skillsets(self) -> None:
-        if self.skillset_fields:
-            del self.blocks[VolunteerBlockIndex.SELECTED_SKILLSETS]["fields"]
+        self.skillset_field_text = " "
 
     def validate_self(self):
         if not self.skillsets: