refactor: updated logic in conditions

Author: canerakdas

.github/workflows/build.yml

@@ -128,6 +128,9 @@ jobs:
           NODE_OPTIONS: '--max_old_space_size=4096'
           # Used for API requests that require GitHub API scopes
           NEXT_GITHUB_API_KEY: ${{ secrets.GITHUB_TOKEN }}
+          # We want to ensure that static exports for all locales are triggered only on `push` events to save resources
+          # and time.
+          NEXT_PUBLIC_STATIC_EXPORT_LOCALE: true
 
       - name: Build Next.js (Static Default Locale)
         # We want to generate static output in the default language within Pull Requests
@@ -146,8 +149,8 @@ jobs:
           NODE_OPTIONS: '--max_old_space_size=4096'
           # Used for API requests that require GitHub API scopes
           NEXT_GITHUB_API_KEY: ${{ secrets.GITHUB_TOKEN }}
-          # We want to ensure that the static export for default locale only happens on `pull_request_target` events
-          NEXT_PUBLIC_STATIC_EXPORT_LOCALE: true
+          # We want to ensure that static exports for all locales do not occur on `pull_request_target` events
+          NEXT_PUBLIC_STATIC_EXPORT_LOCALE: false
 
       - name: Sync Orama Cloud
         # We only want to sync the Orama Cloud production indexes on `push` events.

apps/site/app/[locale]/[...path]/page.tsx

@@ -25,8 +25,8 @@ export const generateMetadata = basePage.generateMetadata;
 
 // Generates all possible static paths based on the locales and environment configuration
 // - Returns an empty array if static export is disabled (`ENABLE_STATIC_EXPORT` is false)
-// - If `ENABLE_STATIC_EXPORT_LOCALE` is true, generates paths only for the default locale
-// - Otherwise, generates paths for all available locales
+// - If `ENABLE_STATIC_EXPORT_LOCALE` is true, generates paths for all available locales
+// - Otherwise, generates paths only for the default locale
 // @see https://nextjs.org/docs/app/api-reference/functions/generate-static-params
 export const generateStaticParams = async () => {
   // Return an empty array if static export is disabled
@@ -43,17 +43,13 @@ export const generateStaticParams = async () => {
     );
   };
 
-  // Handles the case where static export is enabled but only for the default locale
-  if (ENABLE_STATIC_EXPORT_LOCALE) {
-    const routes = await getRoutesForLocale(defaultLocale.code);
-
-    return routes.sort();
-  }
+  // Determine which locales to include in the static export
+  const locales = ENABLE_STATIC_EXPORT_LOCALE
+    ? availableLocaleCodes
+    : [defaultLocale.code];
 
   // Generates all possible routes for all available locales
-  const routes = await Promise.all(
-    availableLocaleCodes.map(getRoutesForLocale)
-  );
+  const routes = await Promise.all(locales.map(getRoutesForLocale));
 
   return routes.flat().sort();
 };

apps/site/app/[locale]/page.tsx

@@ -42,8 +42,8 @@ export const generateMetadata = async (props: DynamicParams) => {
 
 // Generates all possible static paths based on the locales and environment configuration
 // - Returns an empty array if static export is disabled (`ENABLE_STATIC_EXPORT` is false)
-// - If `ENABLE_STATIC_EXPORT_LOCALE` is true, generates paths only for the default locale
-// - Otherwise, generates paths for all available locales
+// - If `ENABLE_STATIC_EXPORT_LOCALE` is true, generates paths for all available locales
+// - Otherwise, generates paths only for the default locale
 // @see https://nextjs.org/docs/app/api-reference/functions/generate-static-params
 export const generateStaticParams = async () => {
   // Return an empty array if static export is disabled
@@ -53,8 +53,8 @@ export const generateStaticParams = async () => {
 
   // Determine which locales to include in the static export
   const locales = ENABLE_STATIC_EXPORT_LOCALE
-    ? [defaultLocale.code]
-    : availableLocaleCodes;
+    ? availableLocaleCodes
+    : [defaultLocale.code];
 
   const routes = await Promise.all(
     // Gets all mapped routes to the Next.js Routing Engine by Locale