Handle null people in pairing history

- also try to prevent that from happening

app/routes/project.$projectId.history/load_history.server.ts

@@ -113,7 +113,7 @@ export default async (
       peopleSet.add(localPeople[r.person_id])
     );
 
-    instance.people = Array.from(peopleSet);
+    instance.people = Array.from(peopleSet).filter((p) => p !== null);
     snapshot.pairingInstances[idx] = instance;
     snapshots[pairingTime] = snapshot;
   });

app/routes/project.$projectId/components/history/PairingHistoryRecord.tsx

@@ -1,13 +1,23 @@
 import { DateTime } from "luxon";
 import React, { useContext } from "react";
-import { ProjectPairingSnapshot } from "~/api/common/interfaces/parrit.interfaces";
+import {
+  Person,
+  ProjectPairingSnapshot,
+} from "~/api/common/interfaces/parrit.interfaces";
 import { ProjectContext } from "../../contexts/ProjectContext";
 import { TrashIcon } from "~/ui/TrashIcon";
 
 interface Props {
   pairingArrangement: ProjectPairingSnapshot;
 }
 
+const snip = (person?: Person): string => {
+  if (!person) {
+    return "null";
+  }
+  return person.name.slice(0, 3);
+};
+
 const PairingHistoryRecord: React.FC<Props> = (props) => {
   const { pairingArrangement } = props;
   const formattedDate = DateTime.fromISO(
@@ -20,7 +30,7 @@ const PairingHistoryRecord: React.FC<Props> = (props) => {
     .map((instance) => {
       const pbName = instance.pairingBoardName ?? "null";
       // get the first 3 letters of each person's name
-      const names = instance.people.map((person) => person.name.slice(0, 3));
+      const names = instance.people.map(snip);
       names.push(pbName.slice(0, 3));
       // alphabetize the names
       names.sort();

app/routes/project.$projectId/hydrateProject.server.ts

@@ -141,8 +141,8 @@ export default async ({ request }: LoaderFunctionArgs): Promise<Project> => {
       name: "Floating",
       exempt: false,
     },
-    people: Array.from(peopleMap.values()),
-    roles: Array.from(roleMap.values()),
+    people: Array.from(peopleMap.values()).filter((v) => v !== null),
+    roles: Array.from(roleMap.values()).filter((v) => v !== null),
   };
 
   return project;