Merge pull request #18 from SteakFisher/dropdown-response-modification

dropdown now uses id instead of value
SteakFisher · Jun 11, 2024 · 17bad11 · 17bad11
2 parents b9e1966 + 4c84408
commit 17bad11
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/components/form-display/DropdownComponent.tsx b/src/components/form-display/DropdownComponent.tsx
@@ -30,7 +30,11 @@ export default function DropdownComponent({ item, id } : { id: string; item: For
       </CardHeader>
       <CardContent>
         <Select onValueChange={(value) => {
-          formResponses[id] = { selected: value, type: "dropdown" }
+          props.items.map((item) => {
+            if (item.value === value) {
+              formResponses[id] = { selected: item.id, type: "dropdown" }
+            }
+          })
         }}>
           <SelectTrigger className="w-[180px]">
             <SelectValue placeholder="Select Item" />

diff --git a/src/functions/validations.ts b/src/functions/validations.ts
@@ -143,7 +143,7 @@ function validateDropdownInput(item: FormItem, response: DropdownResponse, error
   if (!response.selected && !item.props.required) return;
 
   // Workaround for zod to work tsk tsk
-  const zodLiterals = item.props.items.map((option) => z.literal(option.value)) as unknown as readonly [ z.ZodLiteral<string>, z.ZodLiteral<string>, ...z.ZodLiteral<string>[] ]
+  const zodLiterals = item.props.items.map((option) => z.literal(option.id)) as unknown as readonly [ z.ZodLiteral<string>, z.ZodLiteral<string>, ...z.ZodLiteral<string>[] ]
 
   let dropdownResponseValidate = z.object({
     selected: z.union(zodLiterals, {