Merge pull request #585 from psarando/notifications-link-fix

Check for null URLs in Notification message links
cyverse-de · Aug 17, 2024 · 3ce0834 · 3ce0834
2 parents 5e65af0 + 58e30be
commit 3ce0834
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/components/notifications/Message.js b/src/components/notifications/Message.js
@@ -37,10 +37,12 @@ import RatingWidget from "components/apps/RatingWidget";
 function MessageLink(props) {
     const { message, href, as } = props;
 
-    return (
+    return href ? (
         <Link href={href} as={as} passHref>
             <DELink text={message} />
         </Link>
+    ) : (
+        message
     );
 }
 
@@ -141,7 +143,7 @@ function DataLink(props) {
     const href =
         path && getFolderPage(rootFolderAction ? path : getParentPath(path));
 
-    return href ? <MessageLink href={href} message={message} /> : message;
+    return <MessageLink href={href} message={message} />;
 }
 
 function TeamLink(props) {