Change page purging to all instead of locale specific

Author: Maypher

src/collections/Media.ts

@@ -59,15 +59,10 @@ export const Media: CollectionConfig = {
         })
 
         if (isThumbnail.totalDocs > 0) {
-          if (req.locale === 'all') {
-            routing.locales.forEach((locale) => {
-              revalidatePath(`/${locale}/projects`)
-              purgeRoute(`${locale}/projects`)
-            })
-          } else {
-            revalidatePath(`/${req.locale}/projects`)
-            purgeRoute(`${req.locale}/projects`)
-          }
+          routing.locales.forEach((locale) => {
+            revalidatePath(`/${locale}/projects`)
+            purgeRoute(`${locale}/projects`)
+          })
         }
       },
     ],

src/collections/Projects.ts

@@ -274,31 +274,24 @@ export const Projects: CollectionConfig = {
   },
   hooks: {
     afterChange: [
-      async ({ doc, req }) => {
+      async ({ doc }) => {
         // Ignore draft updates
         // Only revalidate cache when a document has been published
         if (doc._status === 'published') {
-          const updatedLocale = req.locale
           const updateURL = `/projects/${doc.id}`
 
-          // If all locales were updated then revalidate all languages otherwise only the updated locale
-          if (updatedLocale === 'all')
-            routing.locales.forEach(async (locale) => {
-              revalidatePath(`/${locale}${updateURL}`)
-              await purgeURL(`${locale}${updateURL}`) // Purge nginx cache
-
-              // Revalidate projects page in case name or thumbnail were changed
-              revalidatePath(`/${locale}/projects`)
-              await purgeURL(`${locale}/projects`)
-            })
-          else {
-            revalidatePath(`/${updatedLocale}${updateURL}`)
-            await purgeURL(`${updatedLocale}${updateURL}`)
+          // Revalidate all locales since when making changes not related
+          // to localization it only updates that locale instead of all.
+          // Revalidating everything (even if not needed) is an ease of use
+          // tradeoff.
+          routing.locales.forEach(async (locale) => {
+            revalidatePath(`/${locale}${updateURL}`)
+            await purgeURL(`${locale}${updateURL}`) // Purge nginx cache
 
             // Revalidate projects page in case name or thumbnail were changed
-            revalidatePath(`/${updatedLocale}/projects`)
-            await purgeURL(`${updatedLocale}/projects`)
-          }
+            revalidatePath(`/${locale}/projects`)
+            await purgeURL(`${locale}/projects`)
+          })
         }
       },
     ],

src/globals/Conclusion.ts

@@ -55,18 +55,12 @@ export const Conclusion: GlobalConfig = {
   },
   hooks: {
     afterChange: [
-      async ({ req, doc }) => {
+      async ({ doc }) => {
         if (doc._status === 'published') {
-          const updatedLocale = req.locale
-          if (updatedLocale === 'all')
-            routing.locales.map(async (locale) => {
-              revalidatePath(`/${locale}/conclusion`)
-              await purgeRoute(`${locale}/conclusion`)
-            })
-          else {
-            revalidatePath(`/${updatedLocale}/conclusion`)
-            await purgeRoute(`${updatedLocale}/conclusion`)
-          }
+          routing.locales.map(async (locale) => {
+            revalidatePath(`/${locale}/conclusion`)
+            await purgeRoute(`${locale}/conclusion`)
+          })
         }
       },
     ],

src/globals/MainPage.ts

@@ -228,18 +228,12 @@ const MainPage: GlobalConfig = {
   },
   hooks: {
     afterChange: [
-      async ({ req, doc }) => {
+      async ({ doc }) => {
         if (doc._status === 'published') {
-          const updatedLocale = req.locale
-          if (updatedLocale === 'all')
-            routing.locales.map(async (locale) => {
-              revalidatePath(`/${locale}`)
-              await purgeRoute(`${locale}`)
-            })
-          else {
-            revalidatePath(`/${updatedLocale}`)
-            await purgeRoute(`${updatedLocale}`)
-          }
+          routing.locales.map(async (locale) => {
+            revalidatePath(`/${locale}`)
+            await purgeRoute(`${locale}`)
+          })
         }
       },
     ],

src/globals/Sculptures.ts

@@ -101,18 +101,12 @@ export const Sculptures: GlobalConfig = {
   },
   hooks: {
     afterChange: [
-      async ({ req, doc }) => {
+      async ({ doc }) => {
         if (doc._status === 'published') {
-          const updatedLocale = req.locale
-          if (updatedLocale === 'all')
-            routing.locales.map(async (locale) => {
-              revalidatePath(`/${locale}/sculptures`)
-              await purgeRoute(`${locale}/sculptures`)
-            })
-          else {
-            revalidatePath(`/${updatedLocale}/sculptures`)
-            await purgeRoute(`${updatedLocale}/sculptures`)
-          }
+          routing.locales.map(async (locale) => {
+            revalidatePath(`/${locale}/sculptures`)
+            await purgeRoute(`${locale}/sculptures`)
+          })
         }
       },
     ],