refactor: remove unnecessary QueryProvider wrapper and optimize layout structure

Robert27 · Robert27 · commit 63954fabac39 · 2025-12-08T01:17:12.000+01:00
diff --git a/frontend/app/(app)/layout.tsx b/frontend/app/(app)/layout.tsx
@@ -1,5 +1,4 @@
 import { DashboardSidebar } from '@/components/dashboard-sidebar'
-import { QueryProvider } from '@/components/query-provider'
 import { SidebarProvider } from '@/components/ui/sidebar'
 import { Toaster } from '@/components/ui/sonner'
 import { UnifiedFooter } from '@/components/unified-footer'
@@ -10,15 +9,13 @@ export default async function AppLayout({
 }: Readonly<{ children: React.ReactNode }>) {
 	await requireUser()
 	return (
-		<QueryProvider>
-			<SidebarProvider>
-				<DashboardSidebar />
-				<main className="flex-1 flex flex-col min-h-screen">
-					<div className="flex-1">{children}</div>
-					<UnifiedFooter variant="app" />
-				</main>
-			</SidebarProvider>
+		<SidebarProvider>
+			<DashboardSidebar />
+			<main className="flex-1 flex flex-col min-h-screen">
+				<div className="flex-1">{children}</div>
+				<UnifiedFooter variant="app" />
+			</main>
 			<Toaster />
-		</QueryProvider>
+		</SidebarProvider>
 	)
 }
diff --git a/frontend/app/e/layout.tsx b/frontend/app/e/layout.tsx
@@ -1,5 +1,4 @@
 import type { Metadata } from 'next'
-import { QueryProvider } from '@/components/query-provider'
 
 export const metadata: Metadata = {
 	title: 'Campus Life Events',
@@ -11,11 +10,5 @@ export default function PublicLayout({
 }: {
 	children: React.ReactNode
 }) {
-	return (
-		<html lang="de">
-			<body className="antialiased">
-				<QueryProvider>{children}</QueryProvider>
-			</body>
-		</html>
-	)
+	return <>{children}</>
 }
diff --git a/frontend/app/layout.tsx b/frontend/app/layout.tsx
@@ -1,5 +1,6 @@
 import type { Metadata } from 'next'
 import './globals.css'
+import { QueryProvider } from '@/components/query-provider'
 import { ThemeProvider } from '@/components/theme-provider'
 
 export const metadata: Metadata = {
@@ -36,7 +37,9 @@ export default function RootLayout({
 	return (
 		<html lang="de" suppressHydrationWarning>
 			<body className="antialiased min-h-screen bg-background text-foreground">
-				<ThemeProvider>{children}</ThemeProvider>
+				<QueryProvider>
+					<ThemeProvider>{children}</ThemeProvider>
+				</QueryProvider>
 			</body>
 		</html>
 	)
diff --git a/frontend/components/query-provider.tsx b/frontend/components/query-provider.tsx
@@ -11,6 +11,7 @@ export function QueryProvider({ children }: { children: React.ReactNode }) {
 				defaultOptions: {
 					queries: {
 						staleTime: 60 * 1000,
+						gcTime: 5 * 60 * 1000,
 						retry: 1
 					}
 				}
diff --git a/frontend/components/unified-footer.tsx b/frontend/components/unified-footer.tsx
@@ -1,3 +1,5 @@
+'use client'
+
 import Link from 'next/link'
 import { ThemeToggle } from '@/components/theme-toggle'
 
diff --git a/frontend/next.config.ts b/frontend/next.config.ts
@@ -13,16 +13,19 @@ const nextConfig: NextConfig = {
 				},
 			];
 		}
-		// In production, let ingress handle the API routing
 		return [];
 	},
 	output: "standalone",
+	cacheComponents: true,
 	experimental: {
 		webpackMemoryOptimizations: true,
+		staleTimes: {
+			dynamic: 30,
+			static: 180,
+		},
 	},
-	reactCompiler: true,
+	reactCompiler: false,
 	poweredByHeader: false,
-
 };
 
 export default nextConfig;

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@ export function QueryProvider({ children }: { children: React.ReactNode }) {`
`11`	`11`	`defaultOptions: {`
`12`	`12`	`queries: {`
`13`	`13`	`staleTime: 60 * 1000,`
	`14`	`+ gcTime: 5 * 60 * 1000,`
`14`	`15`	`retry: 1`
`15`	`16`	`}`
`16`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+'use client'`
	`2`	`+`
`1`	`3`	`import Link from 'next/link'`
`2`	`4`	`import { ThemeToggle } from '@/components/theme-toggle'`
`3`	`5`