Properly return values from list form item

Author: Jurredr

src/components/chat-item/chat-item-form-items.ts

@@ -348,8 +348,8 @@ export class ChatItemFormItemsWrapper {
     this.onAllFormItemsDisabled?.();
   };
 
-  getAllValues = (): Record<string, string> => {
-    const valueMap: Record<string, string> = {};
+  getAllValues = (): Record<string, string | string[]> => {
+    const valueMap: Record<string, string | string[]> = {};
     Object.keys(this.options).forEach(chatOptionId => {
       valueMap[chatOptionId] = this.options[chatOptionId].getValue();
     });

src/components/chat-item/prompt-input/prompt-options.ts

@@ -77,7 +77,7 @@ export class PromptOptions {
     });
   };
 
-  public readonly getOptionValues = (): Record<string, string> => {
+  public readonly getOptionValues = (): Record<string, string | string[]> => {
     return this.formItemsWrapper?.getAllValues() ?? {};
   };
 }

src/components/feedback-form/feedback-form.ts

@@ -162,7 +162,7 @@ export class FeedbackForm {
     chatItem: Partial<ChatItem>;
     title?: string;
     description?: string;
-    onFormAction?: (actionName: string, formData: Record<string, string>) => void;
+    onFormAction?: (actionName: string, formData: Record<string, string | string[]>) => void;
     onFormDisabled?: () => void;
     onCloseButtonClick?: (e: Event) => void;
   }): Array<ExtendedHTMLElement | HTMLElement | string | DomBuilderObject> => {

src/components/form-items/form-item-list.ts

@@ -21,13 +21,13 @@ export interface FormItemListProps {
   label?: HTMLElement | ExtendedHTMLElement | string;
   description?: ExtendedHTMLElement;
   wrapperTestId?: string;
-  onChange?: (values: string) => void;
+  onChange?: (values: string[]) => void;
 }
 
 export abstract class FormItemListAbstract {
   render: ExtendedHTMLElement;
   setValue = (value: ListItemEntry[]): void => { };
-  getValue = (): string => '';
+  getValue = (): string[] => [];
   setEnabled = (enabled: boolean): void => { };
 }
 
@@ -176,9 +176,8 @@ export class FormItemListInternal extends FormItemListAbstract {
     }
   };
 
-  getValue = (): string => {
-    // TODO: Implement this properly
-    return Object.values(this.formData).join(',');
+  getValue = (): string[] => {
+    return Object.values(this.formData);
   };
 
   setEnabled = (enabled: boolean): void => {
@@ -195,6 +194,6 @@ export class FormItemList extends FormItemListAbstract {
   }
 
   setValue = (value: ListItemEntry[]): void => { };
-  getValue = (): string => '';
+  getValue = (): string[] => [];
   setEnabled = (enabled: boolean): void => { };
 }

src/static.ts

@@ -489,7 +489,7 @@ export interface ChatPrompt {
   prompt?: string;
   escapedPrompt?: string;
   command?: string;
-  options?: Record<string, string>;
+  options?: Record<string, string | string[]>;
   context?: string[] | QuickActionCommand[];
 }