From b24e1855e97361df837a52aded29d4e5c222c38b Mon Sep 17 00:00:00 2001
From: Jake Laderman <jsladerman@gmail.com>
Date: Thu, 19 Sep 2024 12:22:53 -0400
Subject: [PATCH] add clickable urls to notifications

---
 assets/package.json                           |  2 +-
 .../cd/clusters/ClusterUpgradeFlyover.tsx     | 14 -------------
 .../components/notifications/Notification.tsx |  9 ++++----
 assets/src/utils/textWithUrls.tsx             | 21 +++++++++++++++++++
 assets/yarn.lock                              | 10 ++++-----
 5 files changed, 32 insertions(+), 24 deletions(-)
 create mode 100644 assets/src/utils/textWithUrls.tsx

diff --git a/assets/package.json b/assets/package.json
index 20fa9e547b..262ffdd9d4 100644
--- a/assets/package.json
+++ b/assets/package.json
@@ -46,7 +46,7 @@
     "@nivo/pie": "0.87.0",
     "@nivo/radial-bar": "0.87.0",
     "@nivo/tooltip": "0.87.0",
-    "@pluralsh/design-system": "3.67.2",
+    "@pluralsh/design-system": "3.68.0",
     "@react-hooks-library/core": "0.6.0",
     "@saas-ui/use-hotkeys": "1.1.3",
     "@tanstack/react-virtual": "3.0.1",
diff --git a/assets/src/components/cd/clusters/ClusterUpgradeFlyover.tsx b/assets/src/components/cd/clusters/ClusterUpgradeFlyover.tsx
index beef4564a5..e68ea87c3f 100644
--- a/assets/src/components/cd/clusters/ClusterUpgradeFlyover.tsx
+++ b/assets/src/components/cd/clusters/ClusterUpgradeFlyover.tsx
@@ -419,21 +419,7 @@ export function ClusterUpgradeFlyover({
       header={`Upgrade Plan for ${cluster?.name}`}
       open={open}
       onClose={onClose}
-      portal
-      size="large"
       minWidth={920}
-      actions={
-        <Button
-          type="button"
-          secondary
-          onClick={(e) => {
-            e.preventDefault()
-            onClose()
-          }}
-        >
-          Cancel
-        </Button>
-      }
     >
       <FlyoverContent
         open={open}
diff --git a/assets/src/components/notifications/Notification.tsx b/assets/src/components/notifications/Notification.tsx
index 5db4cdec47..b73b4bfb90 100644
--- a/assets/src/components/notifications/Notification.tsx
+++ b/assets/src/components/notifications/Notification.tsx
@@ -1,7 +1,8 @@
-import { useTheme } from 'styled-components'
-import React, { useEffect, useRef, useState } from 'react'
 import { Button } from '@pluralsh/design-system'
 import moment from 'moment/moment'
+import { useEffect, useRef, useState } from 'react'
+import { useTheme } from 'styled-components'
+import { renderTextWithUrls } from 'utils/textWithUrls'
 
 import { AppNotificationFragment } from '../../generated/graphql'
 
@@ -43,7 +44,6 @@ export default function Notification({
         flexDirection: 'column',
         gap: theme.spacing.small,
         padding: `${theme.spacing.large}px ${theme.spacing.medium}px `,
-        width: '100%',
       }}
     >
       <div
@@ -61,6 +61,7 @@ export default function Notification({
       <div
         ref={contentRef}
         css={{
+          wordBreak: 'break-word',
           ...(expand
             ? {}
             : {
@@ -71,7 +72,7 @@ export default function Notification({
               }),
         }}
       >
-        {notification.text}
+        {renderTextWithUrls(notification.text ?? '')}
       </div>
       {clamped && (
         <Button
diff --git a/assets/src/utils/textWithUrls.tsx b/assets/src/utils/textWithUrls.tsx
new file mode 100644
index 0000000000..f7444aa011
--- /dev/null
+++ b/assets/src/utils/textWithUrls.tsx
@@ -0,0 +1,21 @@
+import { InlineLink } from 'components/utils/typography/InlineLink'
+
+export const renderTextWithUrls = (text: string) => {
+  const urlRegex = /(https?:\/\/[^\s]+)/g
+  const parts = text.split(urlRegex)
+
+  return parts.map((part, index) =>
+    urlRegex.test(part) ? (
+      <InlineLink
+        key={index}
+        href={part}
+        target="_blank"
+        rel="noopener noreferrer"
+      >
+        {part}
+      </InlineLink>
+    ) : (
+      part
+    )
+  )
+}
diff --git a/assets/yarn.lock b/assets/yarn.lock
index 4868349a22..8c9a9ddbb9 100644
--- a/assets/yarn.lock
+++ b/assets/yarn.lock
@@ -4274,9 +4274,9 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@pluralsh/design-system@npm:3.67.2":
-  version: 3.67.2
-  resolution: "@pluralsh/design-system@npm:3.67.2"
+"@pluralsh/design-system@npm:3.68.0":
+  version: 3.68.0
+  resolution: "@pluralsh/design-system@npm:3.68.0"
   dependencies:
     "@floating-ui/react-dom-interactions": 0.13.3
     "@loomhq/loom-embed": 1.5.0
@@ -4324,7 +4324,7 @@ __metadata:
     react-dom: ">=18.3.1"
     react-transition-group: ">=4.4.5"
     styled-components: ">=5.3.11"
-  checksum: f84cfc700e3b467ba71fba40a66e1c81e4e326bdabfb19cb523aff5a8979bf2e3c4d73cbc12afea93368ccecf328485957148b4654a5b60bf5c56546687da2f1
+  checksum: 6461e1534d9d9f1ebe9ad9c1e4672a674458b4b4b672af86018974d4bafa1f2f2bda2166392269cd316248cbb10d24499f8172314e3c1612d6f91b18ab6a0b2c
   languageName: node
   linkType: hard
 
@@ -10112,7 +10112,7 @@ __metadata:
     "@nivo/pie": 0.87.0
     "@nivo/radial-bar": 0.87.0
     "@nivo/tooltip": 0.87.0
-    "@pluralsh/design-system": 3.67.2
+    "@pluralsh/design-system": 3.68.0
     "@pluralsh/eslint-config-typescript": 2.5.150
     "@pluralsh/stylelint-config": 2.0.10
     "@react-hooks-library/core": 0.6.0