fix: fix aggrid table actions

windmill-labs · Apr 30, 2024 · c1f582f · c1f582f
1 parent 0cd7ab4
commit c1f582f
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 12 deletions.
diff --git a/frontend/src/lib/components/apps/components/display/table/AppAggridExplorerTable.svelte b/frontend/src/lib/components/apps/components/display/table/AppAggridExplorerTable.svelte
@@ -236,10 +236,13 @@
 			})
 		}
 
-		return r.map((fields) => ({
-			...fields,
-			cellRenderer: defaultCellRenderer(fields.cellRendererType)
-		}))
+		return r.map((fields) => {
+			let cr = defaultCellRenderer(fields.cellRendererType)
+			return {
+				...fields,
+				...(cr ? { cellRenderer: cr } : {})
+			}
+		})
 	}
 
 	let firstRow: number = 0

diff --git a/frontend/src/lib/components/apps/components/display/table/AppAggridTable.svelte b/frontend/src/lib/components/apps/components/display/table/AppAggridTable.svelte
@@ -38,6 +38,7 @@
 	import { cellRendererFactory, defaultCellRenderer } from './utils'
 	import Popover from '$lib/components/Popover.svelte'
 	import { Button } from '$lib/components/common'
+	import de from 'date-fns/locale/de'
 
 	// import 'ag-grid-community/dist/styles/ag-theme-alpine-dark.css'
 
@@ -255,10 +256,13 @@
 					eGui,
 					{
 						rowData: value,
-						columnDefs: columnDefs.map((fields) => ({
-							...fields,
-							cellRenderer: defaultCellRenderer(fields.cellRendererType)
-						})),
+						columnDefs: columnDefs.map((fields) => {
+							let cr = defaultCellRenderer(fields.cellRendererType)
+							return {
+								...fields,
+								...(cr ? { cellRenderer: cr } : {})
+							}
+						}),
 						pagination: resolvedConfig?.pagination,
 						paginationAutoPageSize: resolvedConfig?.pagination,
 						suppressPaginationPanel: true,
@@ -383,10 +387,13 @@
 
 			api?.updateGridOptions({
 				rowData: value,
-				columnDefs: columnDefs.map((fields) => ({
-					...fields,
-					cellRenderer: defaultCellRenderer(fields.cellRendererType)
-				})),
+				columnDefs: columnDefs.map((fields) => {
+					let cr = defaultCellRenderer(fields.cellRendererType)
+					return {
+						...fields,
+						...(cr ? { cellRenderer: cr } : {})
+					}
+				}),
 				pagination: resolvedConfig?.pagination,
 				paginationAutoPageSize: resolvedConfig?.pagination,
 				suppressPaginationPanel: true,