Merge pull request #230 from dotCMS/fixing-isr-caching

fixing isr caching

Author: wezell

app/docs/[slug]/page.js

@@ -1,5 +1,8 @@
 import { notFound } from "next/navigation";
 import { getDotCMSPage } from "@/util/getDotCMSPage";
+
+// ISR: Revalidate pages every 60 seconds
+export const revalidate = 60;
 import { getSideNav } from "@/services/docs/getSideNav"
 import { isGitHubDoc, getGitHubConfig } from "@/config/github-docs";
 import { getDocsContentWithGitHub } from "@/services/docs/getGitHubContent";

services/docs/getNavSections.ts

@@ -47,9 +47,9 @@ export async function getNavSections(options: FetchOptions = {}): Promise<NavSec
     const res = await fetch(url, {
       method: 'GET',
       headers: Config.Headers,
-      // Ensure server-side request cache doesn't interfere; we manage our own cache
-      cache: 'no-store',
-      next: { revalidate: 0 },
+      // Use Next.js data cache with revalidation (works in serverless)
+      // In-memory navCache above provides request-level deduplication
+      next: { revalidate: ttlSeconds },
     });
 
     if (!res.ok) {

services/renders/getGraphQLPageQuery/getGraphQLPageQuery.ts

@@ -94,7 +94,7 @@ export const getGraphQLPageQuery = async ({
         method: 'POST',
         headers: Config.Headers,
         body: JSON.stringify({ query }),
-        cache: 'no-cache'
+        next: { revalidate: 60 }  // ISR: revalidate every 60 seconds
       }),
       'getGraphQLPageQuery'
     );

util/dotcmsClient.ts

@@ -15,7 +15,6 @@ export const client = createDotCMSClient({
     authToken: authToken,
     siteId: "173aff42881a55a562cec436180999cf",
     requestOptions: {
-        // In production you might want to deal with this differently
-        cache: "no-cache",
+        next: { revalidate: 60 }  // ISR: revalidate cached data every 60 seconds
     }
 });