fix(csv parse): only accept datetimes that aren't numbers (#2749)

Co-authored-by: Henry Fontanier <[email protected]>
dust-tt · Dec 1, 2023 · 5fa3aa3 · 5fa3aa3
1 parent e4d8a66
commit 5fa3aa3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/front/pages/api/w/[wId]/data_sources/[name]/databases/csv.ts b/front/pages/api/w/[wId]/data_sources/[name]/databases/csv.ts
@@ -265,6 +265,8 @@ async function rowsFromCsv(
     // We keep the parsed values from the first parser that succeeds for all non-null values in the column.
     parsedValuesByCol[col] = (() => {
       for (const parser of [
+        // number
+        (v: string) => (isNaN(parseFloat(v)) ? undefined : parseFloat(v)),
         // date/datetime
         (v: string) => {
           const date = new Date(v);
@@ -276,8 +278,6 @@ async function rowsFromCsv(
                 epoch,
               };
         },
-        // number
-        (v: string) => (isNaN(parseFloat(v)) ? undefined : parseFloat(v)),
         // bool
         (v: string) => {
           const lowerV = v.toLowerCase();