fix: dont show rows without id as "old"

Author: Amelia Wattenberger

src/components/HtmlHistogram.tsx

@@ -30,7 +30,6 @@ export function HtmlHistogram(props: HistogramProps) {
   const height = 30;
 
   const { bins } = useMemo(() => {
-    console.log({ original, maxWidth, value })
     const maxBins = maxWidth
       ? Math.max(0, Math.floor(maxWidth / 6) * 0.55)
       : 11;

src/store.ts

@@ -199,12 +199,12 @@ export const createGridStore = () =>
             const modifiedFields = columnNames.filter((columnName) => {
               const type = draft.cellTypes[columnName];
               const oldValue =
-                type === 'date' ? d[columnName].toString() : d[columnName];
+                type === 'date' ? d[columnName]?.toString?.() : d[columnName];
               const newD = diffDataMap.get(id);
               const newValue =
                 type === 'date'
                   ? // @ts-ignore
-                    newD[columnName].toString()
+                    newD[columnName]?.toString?.()
                   : // @ts-ignore
                     newD[columnName];
               return type === 'object'
@@ -222,7 +222,10 @@ export const createGridStore = () =>
           });
           const oldData = parseData(
             diffData
-              .filter((d) => !newDataMap.get(d[idColumnName + '']))
+              .filter(
+                (d) =>
+                  !newDataMap.get(d[idColumnName + '']) && d[idColumnName || '']
+              )
               .map((d) => ({ ...d, __status__: 'old' })),
             draft.cellTypes
           );