fix sorting in Person table in course planning (#105)

UMN-LATIS · Feb 19, 2024 · 918637d · 918637d
1 parent e89097b
commit 918637d
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/resources/js/pages/CoursePlanningPage/stores/usePersonStore.ts b/resources/js/pages/CoursePlanningPage/stores/usePersonStore.ts
@@ -2,8 +2,9 @@ import { defineStore } from "pinia";
 import { computed, reactive, toRefs } from "vue";
 import * as api from "@/api";
 import * as T from "@/types";
-import { countBy, groupBy, keyBy } from "lodash";
+import { countBy, keyBy } from "lodash";
 import { useEnrollmentStore } from "./useEnrollmentStore";
+import { sortByName } from "@/utils";
 
 interface PersonStoreState {
   activeGroupId: T.Group["id"] | null;
@@ -21,10 +22,12 @@ export const usePersonStore = defineStore("person", () => {
   });
 
   const getters = {
-    allPeople: computed(
-      (): T.Person[] =>
-        Object.values(state.personLookupByEmpId).filter(Boolean) as T.Person[],
-    ),
+    allPeople: computed((): T.Person[] => {
+      const people = Object.values(state.personLookupByEmpId).filter(
+        Boolean,
+      ) as T.Person[];
+      return people.sort(sortByName);
+    }),
     getPersonByEmplId: computed(
       () =>
         (emplId: T.Person["emplid"]): T.Person | null => {