Use empty string for strings field

Author: triozer

plugins/google-sheets/src/App.tsx

@@ -175,10 +175,13 @@ export function App({ pluginContext }: AppProps) {
             sheetTitle,
             fields,
             // Determine if the field type is already configured, otherwise default to "string"
-            colFieldTypes: headerRow.map(colName => {
+            colFieldTypes: headerRow.reduce((acc, colName) => {
                 const field = fields.find(field => field?.name === colName)
-                return field?.type ?? "string"
-            }),
+                return {
+                    ...acc,
+                    [colName]: field?.type ?? "string",
+                }
+            }, {}),
         }).then(() => framer.closePlugin())
     }, [context, shouldSyncOnly])
 

plugins/google-sheets/src/pages/MapSheetFields.tsx

@@ -226,7 +226,13 @@ export function MapSheetFieldsPage({
             fields: allFields,
             spreadsheetId,
             sheetTitle,
-            colFieldTypes: fieldConfig.map(field => field.type ?? "string"),
+            colFieldTypes: fieldConfig.reduce(
+                (acc, field) => ({
+                    ...acc,
+                    [field.id]: field.type,
+                }),
+                {}
+            ),
             ignoredColumns: Array.from(disabledColumns),
             slugColumn,
             lastSyncedTime: getLastSyncedTime(pluginContext, slugColumn),

plugins/google-sheets/src/sheets.ts

@@ -258,7 +258,7 @@ export interface SyncResult extends SyncStatus {
 }
 
 interface ProcessSheetRowParams {
-    fieldTypes: CollectionFieldType[]
+    fieldTypes: Record<string, CollectionFieldType>
     row: Row
     rowIndex: number
     uniqueHeaderRowNames: string[]
@@ -275,7 +275,7 @@ export interface SyncMutationOptions {
     fields: ManagedCollectionField[]
     slugColumn: string | null
     ignoredColumns: string[]
-    colFieldTypes: CollectionFieldType[]
+    colFieldTypes: Record<string, CollectionFieldType>
     lastSyncedTime: string | null
 }
 
@@ -331,7 +331,11 @@ function getFieldValue(fieldType: CollectionFieldType, cellValue: CellValue) {
         case "formattedText":
         case "color":
         case "string": {
-            return String(cellValue)
+            if (typeof cellValue === "string") {
+                return cellValue
+            }
+
+            return ""
         }
         default:
             return null
@@ -355,18 +359,7 @@ function processSheetRow({
     for (const [colIndex, cell] of row.entries()) {
         if (ignoredFieldColumnIndexes.includes(colIndex)) continue
 
-        // +1 as zero-indexed, another +1 to account for header row
-        const location = columnToLetter(colIndex + 1) + (rowIndex + 2)
-
-        const fieldValue = getFieldValue(fieldTypes[colIndex], cell)
-
-        if (fieldValue === null) {
-            status.warnings.push({
-                rowIndex,
-                message: `Invalid cell value at ${location}.`,
-            })
-            continue
-        }
+        const fieldValue = getFieldValue(fieldTypes[uniqueHeaderRowNames[colIndex]], cell)
 
         if (colIndex === slugFieldColumnIndex) {
             if (typeof fieldValue !== "string") {
@@ -394,7 +387,11 @@ function processSheetRow({
 
     for (const headerRowName of uniqueHeaderRowNames) {
         if (!(headerRowName in fieldData)) {
-            fieldData[headerRowName] = null
+            if (["string", "formattedText"].includes(fieldTypes[headerRowName])) {
+                fieldData[headerRowName] = ""
+            } else {
+                fieldData[headerRowName] = null
+            }
         }
     }