Merge pull request #81 from skorphil/69-add-form-submit-success

implemented successful submit toast
skorphil · Mar 12, 2024 · ccd26c3 · ccd26c3
2 parents 7db4240 + 8df3ef6
commit ccd26c3
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 10 deletions.
diff --git a/components/InstitutionTab/InstitutionTab.jsx b/components/InstitutionTab/InstitutionTab.jsx
@@ -16,7 +16,8 @@ export const InstitutionTab = forwardRef(
     const institutionFields =
       institutionsFieldArray.fields[getInstitutionIndex(institutionName)];
     const institutionDefaultValues =
-      defaultValues.institutions[getInstitutionIndex(institutionName)];
+      defaultValues.institutions?.[getInstitutionIndex(institutionName)] ??
+      null;
     const institutionCurrentValues = useWatch({
       control,
       name: institutionName,

diff --git a/components/RecordForm/RecordForm.jsx b/components/RecordForm/RecordForm.jsx
@@ -7,7 +7,7 @@ Root element of the form
 import { useForm, FormProvider, useFieldArray } from "react-hook-form";
 import classes from "./RecordForm.module.css";
 import { Button, Progress, useToast } from "@chakra-ui/react";
-import { useState, useEffect } from "react";
+import { useState } from "react";
 
 import { InstitutionsList } from "components/InstitutionsList";
 import { FormHeader } from "~/FormHeader";
@@ -22,7 +22,7 @@ export function RecordForm() {
   const [isInstitutionOpen, setIsInstitutionOpen] = useState(false);
   const [selectedInstitutionIndex, setSelectedInstitutionIndex] = useState(0);
   const [errorState, setErrorState] = useState(false);
-
+  const toast = useToast({ position: "top" });
   const router = useRouter();
   const arrayName = "institutions";
 
@@ -88,8 +88,23 @@ export function RecordForm() {
                 Cancel
               </Button>
               <Button
-                onClick={formMethods.handleSubmit((data) => {
-                  appendRecord(data);
+                onClick={formMethods.handleSubmit(async (data) => {
+                  try {
+                    await appendRecord(data);
+                    router.push("/");
+                    toast({
+                      title: "Record saved",
+                      status: "success",
+                      duration: 4000,
+                    });
+                  } catch (error) {
+                    toast({
+                      title: "Error saving record",
+                      description: error.message,
+                      status: "error",
+                      duration: 4000,
+                    });
+                  }
                 })}
               >
                 Save

diff --git a/public/next.svg b/public/next.svg
diff --git a/public/vercel.svg b/public/vercel.svg
diff --git a/serverActions/appendRecord.js b/serverActions/appendRecord.js
@@ -99,6 +99,6 @@ export async function appendRecord(formData) {
     await record.save();
     console.log("Document saved to db");
   } catch (error) {
-    throw new Error("Error saving document:", error);
+    throw new Error("Error saving document:", error.message);
   }
 }
diff --git a/serverActions/getLatestRecord.js b/serverActions/getLatestRecord.js
@@ -19,15 +19,15 @@ export async function getLatestRecord() {
       .sort({ createdAt: -1 })
       .limit(1)
       .exec();
-    const institutionsList = latestRecord[0].institutions.toObject({
+    const institutionsList = latestRecord[0]?.institutions.toObject({
       transform: function (doc, ret) {
         delete ret._id;
         return ret;
       },
     });
 
     const initialValues = {
-      institutions: institutionsList.map((institution) => ({
+      institutions: institutionsList?.map((institution) => ({
         ...institution,
         isDeleted: false,
       })),