Merge pull request #2373 from graphcommerce-org/feature/redirect-lega…

…cy-html-suffix Handle the case where one switches from using the default .html URL suffix, to not using one
graphcommerce-org · Sep 19, 2024 · e941ee4 · e941ee4
2 parents a973f21 + 53ffc91
commit e941ee4
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/.changeset/healthy-lies-act.md b/.changeset/healthy-lies-act.md
@@ -0,0 +1,5 @@
+---
+'@graphcommerce/magento-store': minor
+---
+
+Handle the case where one switches from using the default .html URL suffix, to not using one
diff --git a/packages/magento-store/utils/redirectOrNotFound.ts b/packages/magento-store/utils/redirectOrNotFound.ts
@@ -68,6 +68,11 @@ export async function redirectOrNotFound(
       candidates.add(from.endsWith(suffix) ? from.slice(0, -suffix.length) : `${from}${suffix}`)
     })
 
+    // Handle the case where we transition from using the default .html suffix, to not using one
+    if (from.endsWith('.html')) {
+      candidates.add(from.slice(0, -('.html'.length)))
+    }
+
     const routePromises = [...candidates].filter(Boolean).map(
       async (url) =>
         (