Skip to content

Commit e140f62

Browse files
committed
fix: remove dependency on builtInDashboardList to fix typecheck
1 parent 7c0bd32 commit e140f62

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

apps/webapp/app/routes/api.v1.query.dashboards._index.ts

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,9 @@ import { json } from "@remix-run/server-runtime";
22
import type { DashboardSummary, DashboardWidgetSummary } from "@trigger.dev/core/v3/schemas";
33
import type { BuiltInDashboard } from "~/presenters/v3/MetricDashboardPresenter.server";
44
import { createLoaderApiRoute } from "~/services/routeBuilders/apiBuilder.server";
5-
import { builtInDashboardList } from "~/presenters/v3/BuiltInDashboards.server";
5+
import { builtInDashboard } from "~/presenters/v3/BuiltInDashboards.server";
6+
7+
const BUILT_IN_DASHBOARD_KEYS = ["overview", "llm"];
68

79
function serializeDashboard(dashboard: BuiltInDashboard): DashboardSummary {
810
const widgets: DashboardWidgetSummary[] = [];
@@ -40,7 +42,13 @@ export const loader = createLoaderApiRoute(
4042
},
4143
},
4244
async () => {
43-
const dashboards = builtInDashboardList().map(serializeDashboard);
45+
const dashboards = BUILT_IN_DASHBOARD_KEYS.map((key) => {
46+
try {
47+
return serializeDashboard(builtInDashboard(key));
48+
} catch {
49+
return null;
50+
}
51+
}).filter((d): d is DashboardSummary => d !== null);
4452
return json({ dashboards });
4553
}
4654
);

0 commit comments

Comments
 (0)