Merge pull request #463 from G-Core/redirect-article-fix

remove duplicated articles from sitemap
G-Core · Oct 9, 2023 · a881346 · a881346
2 parents 8c058b3 + 14ee13d
commit a881346
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/scully/plugins/algolia.ts b/scully/plugins/algolia.ts
@@ -71,7 +71,7 @@ const updateAlgoliaIndex = async (dom, options): Promise<JSDOM> => {
         return dom;
     }
     try {
-        if (options.data.title !== 'metadata') {
+        if (options.data.title !== 'metadata' && options.data.published && !options.data.redirect) {
             const { window } = dom;
             const { document } = window;
             const mainTitle = document.querySelector('h1')?.innerHTML || '';

diff --git a/scully/plugins/sitemap/sitemap.ts b/scully/plugins/sitemap/sitemap.ts
@@ -138,9 +138,9 @@ export const sitemapPlugin = async (routes: Array<HandledRoute>): Promise<void>
 
     routes.forEach((route: HandledRoute) => {
         if (
-            config.ignoredRoutes &&
-            (config.ignoredRoutes.includes(route.route) ||
-                config.ignoredRoutesPattern.some((regExp) => regExp.test(route.route)))
+            route.data.redirect ||
+            config.ignoredRoutes.includes(route.route) ||
+            config.ignoredRoutesPattern.some((regExp) => regExp.test(route.route))
         ) {
             return;
         }