Dynamic label bug fixed

frontend/webapp/components/setup/connection/create.connection.form/dynamic.fields.tsx

@@ -8,6 +8,7 @@ import {
   KeyvalText,
   MultiInput,
 } from '@/design.system';
+import { safeJsonParse } from '@/utils/functions/strings';
 
 export function renderFields(
   fields: Field[],
@@ -55,12 +56,16 @@ export function renderFields(
           </FieldWrapper>
         );
       case INPUT_TYPES.MULTI_INPUT:
-        const userInputData = dynamicFields[name]
-          ? JSON.parse(dynamicFields[name])
-          : null;
+        const userInputData = safeJsonParse<string[] | null>(
+          dynamicFields[name],
+          null
+        );
 
+        // Use safeJsonParse to parse field?.initial_value, defaulting to an empty string if not available.
+        // This assumes that the initial value is supposed to be a string when parsed successfully.
+        // Adjust the fallback value as necessary to match the expected type
         const initialList =
-          userInputData || JSON.parse(field?.initial_value || '');
+          userInputData || safeJsonParse<string[]>(field?.initial_value, []);
 
         return (
           <FieldWrapper key={name}>

frontend/webapp/hooks/sources/useSources.ts

@@ -12,7 +12,7 @@ export function useSources() {
 
   function upsertSources({ sectionData, onSuccess, onError }) {
     const sourceNamesSet = new Set(
-      sources.map((source: SelectedSources) => source.name)
+      sources?.map((source: SelectedSources) => source.name)
     );
     const updatedSectionData: SelectedSources = {};
 

frontend/webapp/utils/functions/strings.tsx

@@ -1,3 +1,14 @@
 export function capitalizeFirstLetter(string: string) {
   return string.charAt(0).toUpperCase() + string.slice(1);
 }
+
+export function safeJsonParse<T>(str: string | undefined, fallback: T): T {
+  if (!str) return fallback;
+  try {
+    const parsed = JSON.parse(str) as T;
+    return parsed;
+  } catch (e) {
+    console.error('Error parsing JSON string:', e);
+    return fallback;
+  }
+}