Merge pull request #6986 from QwikDev/fix-insights-build

fix(insights): fix build
QwikDev · Oct 17, 2024 · 94ee2b4 · 94ee2b4
2 parents cc253b5 + 7e5b7a3
commit 94ee2b4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/packages/insights/src/routes/api/v1/[publicApiKey]/post/index.tsx b/packages/insights/src/routes/api/v1/[publicApiKey]/post/index.tsx
@@ -47,10 +47,10 @@ export const onPost: RequestHandler = async ({ exit, json, request, params }) =>
  }
 };
 
-function cleanupSymbolName(symbolName?: string | null) {
- if (!symbolName) return;
+function cleanupSymbolName(symbolName?: string | null): string | null {
+ if (!symbolName) return null;
  const shortName = symbolName.substring(symbolName.lastIndexOf('_') + 1 || 0);
- if (shortName == 'hW') return;
+ if (shortName == 'hW') return null;
  return shortName;
 }
 function migrate1(payloadJson: any) {

diff --git a/packages/insights/src/routes/app/[publicApiKey]/errors/index.tsx b/packages/insights/src/routes/app/[publicApiKey]/errors/index.tsx
@@ -11,7 +11,6 @@ export const useErrors = routeLoader$(async ({ params }) => {
  .select()
  .from(errorTable)
  .where(eq(errorTable.publicApiKey, params.publicApiKey))
- .limit(1000)
  .orderBy(sql`${errorTable.timestamp} DESC`)
  .limit(1000)
  .all();