very dirty Just Working i18n for most cases

Author: lindsaylevine

lib/pages/getServerSideProps/redirects.js

@@ -1,12 +1,27 @@
 const getNetlifyFunctionName = require("../../helpers/getNetlifyFunctionName");
 const getDataRouteForRoute = require("../../helpers/getDataRouteForRoute");
 const pages = require("./pages");
+const getNextConfig = require("../../helpers/getNextConfig");
 
 const redirects = [];
 
+const nextConfig = getNextConfig();
+
 pages.forEach(({ route, filePath }) => {
   const functionName = getNetlifyFunctionName(filePath);
 
+  // If i18n, need to accommodate for locales
+  if (nextConfig.i18n) {
+    const { locales } = nextConfig.i18n;
+    // Add redirects for each locale
+    locales.forEach((locale) => {
+      redirects.push({
+        route: `/${locale}${route}`,
+        target: `/.netlify/functions/${functionName}`,
+      });
+    });
+  }
+
   // Add one redirect for the page
   redirects.push({
     route,

lib/pages/getStaticProps/redirects.js

@@ -9,6 +9,9 @@ const pages = require("./pages");
 // redirects for handling preview mode.
 const redirects = [];
 
+const defaultLocaleRedirects = [];
+const dynamicLocalesAdded = [];
+
 pages.forEach(({ route, dataRoute, srcRoute }) => {
   const relativePath = getFilePathForRoute(srcRoute || route, "js");
   const filePath = join("pages", relativePath);
@@ -19,23 +22,65 @@ pages.forEach(({ route, dataRoute, srcRoute }) => {
 
   const nextConfig = getNextConfig();
   if (nextConfig.i18n) {
-    const { locales } = nextConfig.i18n;
-    locales.forEach((locale) => {
-      const pageRoute = srcRoute ? route : `${locale}${route}`;
-      const dataRoute_ = srcRoute ? dataRoute : `${locale}${dataRoute}`;
+    const { defaultLocale, locales } = nextConfig.i18n;
+    const isNotDynamic = !srcRoute;
+    if (isNotDynamic) {
+      if (defaultLocale) {
+        redirects.push({
+          route,
+          target: `/${defaultLocale}${route}`,
+        });
+      }
+      locales.forEach((locale) => {
+        const pageRoute = `/${locale}${route}`;
+        redirects.push({
+          route: pageRoute,
+          target,
+          force: true,
+          conditions,
+        });
+      });
+      // Data routes are consistent across locale so it can be outside of the loop
       redirects.push({
-        route: pageRoute,
+        route: dataRoute,
         target,
         force: true,
         conditions,
       });
+    } else {
+      if (defaultLocale && !defaultLocaleRedirects.includes(srcRoute)) {
+        // Add redirect for /normal/:id -> /defaultLocale/normal/:id
+        redirects.push({
+          route: srcRoute,
+          target: `/${defaultLocale}${srcRoute
+            .replace("[", ":")
+            .replace("]", "")}`,
+        });
+        defaultLocaleRedirects.push(srcRoute);
+      }
+      const routeLocale = route.split("/")[1];
+      const srcRouteWithLocale = `/${routeLocale}${srcRoute}`;
+      if (!dynamicLocalesAdded.includes(srcRouteWithLocale)) {
+        // Add one redirect for the page, but only when the NextJS
+        // preview mode cookies are present
+        redirects.push({
+          route: srcRouteWithLocale,
+          target,
+          force: true,
+          conditions,
+        });
+
+        dynamicLocalesAdded.push(srcRouteWithLocale);
+      }
+
+      // Add one redirect for the data route, same conditions
       redirects.push({
-        route: dataRoute_,
+        route: dataRoute,
         target,
         force: true,
         conditions,
       });
-    });
+    }
   } else {
     // Add one redirect for the page, but only when the NextJS
     // preview mode cookies are present

lib/pages/getStaticProps/setup.js

@@ -25,15 +25,16 @@ const setup = ({ functionsPath, publishPath }) => {
 
     const nextConfig = getNextConfig();
 
-    // If an app is using Next 10+'s i18n feature, next will put statically
+    // If an app is using i18n, Next will put statically
     // generated files under a different path for each different locale
+    // i.e. pages/static -> pages/{locale}/static
     if (nextConfig.i18n) {
-      const { locales } = nextConfig.i18n;
-      if (!locales || locales.length === 0) return;
+      const { locales = [] } = nextConfig.i18n;
 
+      // For dynamic routes in i18n, Next auto-prepends with locale
+      // in prerender-manifest; static routes are missing locale so
+      // we need to manually accommodate
       const isNotDynamic = !srcRoute;
-      // Dynamic routes don't need special helper, Next auto prepends with locale
-      // in prerender-manifest, but static routes are missing locale
       if (isNotDynamic) {
         locales.forEach((locale) => {
           // Copy pre-rendered HTML page

lib/pages/getStaticPropsWithRevalidate/redirects.js

@@ -3,24 +3,81 @@ const getFilePathForRoute = require("../../helpers/getFilePathForRoute");
 const getNetlifyFunctionName = require("../../helpers/getNetlifyFunctionName");
 const pages = require("./pages");
 
+const getNextConfig = require("../../helpers/getNextConfig");
+
 const redirects = [];
 
+console.log("REVAL PAGES", pages);
+
+const nextConfig = getNextConfig();
+
+const srcRouteRedirectsAdded = [];
+const dynamicLocalesAdded = [];
+
 pages.forEach(({ route, srcRoute, dataRoute }) => {
   const relativePath = getFilePathForRoute(srcRoute || route, "js");
   const filePath = join("pages", relativePath);
   const functionName = getNetlifyFunctionName(filePath);
+  const target = `/.netlify/functions/${functionName}`;
+
+  if (nextConfig.i18n) {
+    const { defaultLocale, locales } = nextConfig.i18n;
+    const isNotDynamic = !srcRoute;
+    if (isNotDynamic) {
+      if (defaultLocale) {
+        // lol... /reval -> /:function
+        redirects.push({
+          route,
+          target,
+        });
+      }
+      locales.forEach((locale) => {
+        const pageRoute = `/${locale}${route}`;
+        redirects.push({
+          route: pageRoute,
+          target,
+        });
+      });
+      redirects.push({
+        route: dataRoute,
+        target,
+      });
+    } else {
+      // Add a redirect for each srcRoute
+      if (!srcRouteRedirectsAdded.includes(srcRoute)) {
+        redirects.push({
+          route: srcRoute,
+          target,
+        });
+        redirects.push({
+          route: dataRoute,
+          target,
+        });
+        srcRouteRedirectsAdded.push(srcRoute);
+      }
+
+      const routeLocale = route.split("/")[1];
+      if (!dynamicLocalesAdded.includes(routeLocale)) {
+        redirects.push({
+          route: `/${routeLocale}${srcRoute}`,
+          target,
+        });
+        dynamicLocalesAdded.push(routeLocale);
+      }
+    }
+  } else {
+    // Add one redirect for the page
+    redirects.push({
+      route,
+      target,
+    });
 
-  // Add one redirect for the page
-  redirects.push({
-    route,
-    target: `/.netlify/functions/${functionName}`,
-  });
-
-  // Add one redirect for the data route
-  redirects.push({
-    route: dataRoute,
-    target: `/.netlify/functions/${functionName}`,
-  });
+    // Add one redirect for the data route
+    redirects.push({
+      route: dataRoute,
+      target,
+    });
+  }
 });
 
 module.exports = redirects;