Allow filtering by all member statuses in member list view

atst/domain/workspace_roles.py

@@ -9,6 +9,16 @@ from .users import Users
 from .exceptions import NotFoundError
 
 
+MEMBER_STATUSES = [
+    {"name": "active", "display_name": "Active"},
+    {"name": "revoked", "display_name": "Revoked"},
+    {"name": "expired", "display_name": "Invite expired"},
+    {"name": "error", "display_name": "Error on invite"},
+    {"name": "pending", "display_name": "Pending"},
+    {"name": "unknown", "display_name": "Unknown errors"},
+]
+
+
 class WorkspaceRoles(object):
     @classmethod
     def get(cls, workspace_id, user_id):

atst/models/workspace_role.py

@@ -53,11 +53,12 @@ class WorkspaceRole(Base, mixins.TimestampsMixin, mixins.AuditableMixin):
         elif self.latest_invitation:
             if self.latest_invitation.is_revoked:
                 return "Revoked"
-            elif self.latest_invitation.is_rejected_expired:
-                return "Invite expired"
             elif self.latest_invitation.is_rejected_wrong_user:
                 return "Error on invite"
-            elif self.latest_invitation.is_expired:
+            elif (
+                self.latest_invitation.is_rejected_expired
+                or self.latest_invitation.is_expired
+            ):
                 return "Invite expired"
             else:
                 return "Pending"

atst/routes/workspaces.py

@@ -14,7 +14,7 @@ from atst.domain.exceptions import UnauthorizedError, AlreadyExistsError
 from atst.domain.projects import Projects
 from atst.domain.reports import Reports
 from atst.domain.workspaces import Workspaces
-from atst.domain.workspace_roles import WorkspaceRoles
+from atst.domain.workspace_roles import WorkspaceRoles, MEMBER_STATUSES
 from atst.domain.environments import Environments
 from atst.domain.environment_roles import EnvironmentRoles
 from atst.forms.project import NewProjectForm, ProjectForm
@@ -124,6 +124,7 @@ def workspace_members(workspace_id):
         "workspaces/members/index.html",
         workspace=workspace,
         role_choices=WORKSPACE_ROLE_DEFINITIONS,
+        status_choices=MEMBER_STATUSES,
         members=members_list,
         new_member=new_member,
     )

js/components/forms/members_list.js

@@ -11,12 +11,14 @@ const search = (query, members) => {
   }
 }
 
-const filterByStatus = (status, members) => {
+const filterByStatus = (status, statusesByDisplayName, members) => {
+  const getStatusFromDisplayName = (_status) => statusesByDisplayName[_status].name
+
   if (status === '' || status === 'all') {
     return members
   } else {
     return members.filter(
-      (member) => member.status.toLowerCase() === status
+      (member) => getStatusFromDisplayName(member.status) === status
     )
   }
 }
@@ -52,6 +54,7 @@ export default {
   props: {
     members: Array,
     role_choices: Array,
+    status_choices: Array,
   },
 
   data: function () {
@@ -90,6 +93,7 @@ export default {
     return {
       searchValue: '',
       status: '',
+      statusesByDisplayName: indexBy(prop('display_name'), this.status_choices),
       role: '',
       rolesByDisplayName: indexBy(prop('display_name'), this.role_choices),
       sortInfo: {
@@ -104,7 +108,7 @@ export default {
     searchedList: function () {
       return pipe(
         partial(search, [this.searchValue]),
-        partial(filterByStatus, [this.status]),
+        partial(filterByStatus, [this.status, this.statusesByDisplayName]),
         partial(filterByRole, [this.role, this.rolesByDisplayName]),
         partial(sort, [this.sortInfo])
       )(this.members)

templates/workspaces/members/index.html

@@ -47,7 +47,12 @@
 {% endif %}
 
 
-<members-list inline-template id="search-template" v-bind:members='{{ members | tojson}}' v-bind:role_choices='{{ role_choices | tojson}}'>
+<members-list
+  inline-template
+  id="search-template"
+  v-bind:members='{{ members | tojson}}'
+  v-bind:role_choices='{{ role_choices | tojson}}'
+  v-bind:status_choices='{{ status_choices | tojson}}'>
   <div>
   <form class='search-bar' @submit.prevent>
     <div class='usa-input search-input'>
@@ -61,9 +66,9 @@
       <select v-model="status" id="filter-status" name="filter-status">
         <option value="" selected disabled>Filter by status</option>
         <option value="all">View All</option>
-        <option value="active">Active</option>
-        <option value="pending">Pending</option>
-        <option value="denied">Denied</option>
+        {% for status in status_choices %}
+          <option value='{{ status.name }}'>{{ status.display_name }}</option>
+        {% endfor %}
       </select>
     </div>