fix(web): /rss.xml link should be an anchor tag (#17693)

* Make sure that rss.xml is an anchor tag * Remove unneeded ?
island-is · Jan 30, 2025 · 2344d07 · 2344d07
1 parent ae8fb12
commit 2344d07
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/apps/web/components/Organization/Wrapper/OrganizationWrapper.tsx b/apps/web/components/Organization/Wrapper/OrganizationWrapper.tsx
@@ -31,6 +31,7 @@ import {
   Text,
 } from '@island.is/island-ui/core'
 import { theme } from '@island.is/island-ui/theme'
+import { shouldLinkBeAnAnchorTag } from '@island.is/shared/utils'
 import {
   BoostChatPanel,
   boostChatPanelEndpoints,
@@ -1079,12 +1080,12 @@ export const OrganizationWrapper: React.FC<
                   title={navigationData.title}
                   activeItemTitle={activeNavigationItemTitle}
                   renderLink={(link, item) => {
-                    return item?.href ? (
-                      <NextLink href={item?.href} legacyBehavior>
+                    return !item?.href || shouldLinkBeAnAnchorTag(item.href) ? (
+                      link
+                    ) : (
+                      <NextLink href={item.href} legacyBehavior>
                         {link}
                       </NextLink>
-                    ) : (
-                      link
                     )
                   }}
                 />

diff --git a/libs/shared/utils/src/lib/shouldLinkBeAnAnchorTag.ts b/libs/shared/utils/src/lib/shouldLinkBeAnAnchorTag.ts
@@ -4,4 +4,5 @@ import { ProjectBasePath } from '@island.is/shared/constants'
  * then the link should be an anchor tag instead of a nextlink for example
  * */
 export const shouldLinkBeAnAnchorTag = (path: string) =>
-  Object.values(ProjectBasePath).some((basePath) => path.includes(basePath))
+  Object.values(ProjectBasePath).some((basePath) => path.includes(basePath)) ||
+  path.startsWith('/rss.xml')