Merge pull request #70 from whyphi/dev/v0.2

merge: v0.2 to staging
whyphi · Feb 3, 2024 · a9ab37a · a9ab37a
2 parents e07f958 + 8021601
commit a9ab37a
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 6 deletions.
diff --git a/app/admin/listing/[listingId]/insights/page.tsx b/app/admin/listing/[listingId]/insights/page.tsx
@@ -22,7 +22,7 @@ export default function Insights({ params }: { params: { listingId: string } })
   });
 
   // applicantData : list of applicants
-  const [applicantData, setApplicantData] = useState<[] | [Applicant]>([]);
+  const [applicantData, setApplicantData] = useState<[] | Applicant[]>([]);
   // distributionMetrics : object containing frequencies of each metric for all applicants
   const [distributionMetrics, setDistributionMetrics] = useState<DistributionMetricsState>({
     colleges: [],
@@ -54,7 +54,14 @@ export default function Insights({ params }: { params: { listingId: string } })
     fetch(`${process.env.NEXT_PUBLIC_API_BASE_URL}/applicants/${params.listingId}`)
       .then((response) => response.json())
       .then((data: [Applicant]) => {
-        setApplicantData(data)
+        const cleanedData: Applicant[] = data.map((applicant: Applicant) => {
+          return {
+            ...applicant,
+            major: applicant.major.toLowerCase(),
+            minor: applicant.minor.toLowerCase(),
+          };
+        });
+        setApplicantData(cleanedData)
         setIsLoading(false);
         // parseData()
       })

diff --git a/components/Form.tsx b/components/Form.tsx
@@ -140,13 +140,17 @@ export default function Form({ title, questions, listingId, includeEventsAttende
       });
 
       if (response.ok) {
-        // Handle successful response here, e.g., show a success message or redirect
         router.push(`/public/success`);
       } else {
-        // Handle error response here, e.g., show an error message
+        const code = response.status;
+        if (code == 410) {
+          const responseText = await response.text();
+          alert(`Error submitting form: ${responseText}`)
+        } else {
+          console.error('Error submitting form');
+          alert(`Error submitting form. Please contact PCT with a screenshot of the error!`);
+        }
         setIsSubmitting(false);
-        console.error('Error submitting form');
-        alert(`Error submitting form. Please contact PCT with a screenshot of the error!`);
       }
     } catch (error) {
       // Handle any unexpected errors here