Merge pull request #251 from tloncorp/hm/allow-notifications-with-sin…

…gle-contents notification: prevent notification with single piece of content from crashing
tloncorp · Jan 16, 2024 · da1d1fd · da1d1fd
2 parents 1088a24 + 814dfdd
commit da1d1fd
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/ui/src/nav/notifications/Notification.tsx b/ui/src/nav/notifications/Notification.tsx
@@ -142,12 +142,12 @@ const NotificationContent: React.FC<NotificationContent> = ({
   type,
   content,
 }) => {
-  const con = content;
+  const line = (content[1] || '').toString();
   const mentionRe = new RegExp('mentioned');
   const replyRe = new RegExp('replied');
 
-  const isMention = type === 'channel' && mentionRe.test(content[1].toString());
-  const isReply = type === 'channel' && replyRe.test(content[1].toString());
+  const isMention = type === 'channel' && mentionRe.test(line);
+  const isReply = type === 'channel' && replyRe.test(line);
 
   function renderContent(c: YarnContent) {
     if (typeof c === 'string') {