fix: add id field to AppConfigPage DataTable rows
All checks were successful
CI / cleanup-branch (push) Has been skipped
CI / build (push) Successful in 2m51s
CI / docker (push) Successful in 1m9s
CI / deploy-feature (push) Has been skipped
CI / deploy (push) Successful in 35s

DataTable requires rows with an { id: string } constraint. Map
ApplicationConfig to ConfigRow adding id from the application field.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
hsiegeln
2026-03-26 12:55:19 +01:00
parent b0484459a2
commit b1c2950b1e

View File

@@ -6,6 +6,8 @@ import { useAllApplicationConfigs, useUpdateApplicationConfig } from '../../api/
import type { ApplicationConfig } from '../../api/queries/commands';
import styles from './AppConfigPage.module.css';
type ConfigRow = ApplicationConfig & { id: string };
function timeAgo(iso?: string): string {
if (!iso) return '\u2014';
const diff = Date.now() - new Date(iso).getTime();
@@ -45,7 +47,7 @@ export default function AppConfigPage() {
});
}, [updateConfig, toast]);
const columns: Column<ApplicationConfig>[] = useMemo(() => [
const columns: Column<ConfigRow>[] = useMemo(() => [
{
key: '_inspect',
header: '',
@@ -162,9 +164,9 @@ export default function AppConfigPage() {
return (
<div>
<DataTable<ApplicationConfig>
<DataTable<ConfigRow>
columns={columns}
data={configs ?? []}
data={(configs ?? []).map(c => ({ ...c, id: c.application }))}
pageSize={50}
/>
</div>