From 52df3921de3782b9f335d0fe1720374f320720cc Mon Sep 17 00:00:00 2001
From: Noah Santschi-Cooney <noah@santschi-cooney.ch>
Date: Thu, 15 Dec 2022 22:00:37 +0000
Subject: [PATCH] fix new nullability issue

---
 .../executors/secrets/SecretAccessLogsModal.tsx       | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/client/web/src/enterprise/executors/secrets/SecretAccessLogsModal.tsx b/client/web/src/enterprise/executors/secrets/SecretAccessLogsModal.tsx
index 7b23ca8e2a15..6fbb9a7663ec 100644
--- a/client/web/src/enterprise/executors/secrets/SecretAccessLogsModal.tsx
+++ b/client/web/src/enterprise/executors/secrets/SecretAccessLogsModal.tsx
@@ -77,13 +77,10 @@ const ExecutorSecretAccessLogNode: React.FunctionComponent<React.PropsWithChildr
         <div className="d-flex justify-content-between align-items-center flex-wrap mb-0">
             <PersonLink
                 person={{
-                    displayName: node.user.displayName || node.user.username,
-                    email: node.user.email,
-                    user: {
-                        displayName: node.user.displayName,
-                        url: node.user.url,
-                        username: node.user.username,
-                    },
+                    // empty strings are fine here, as they are only used when `user` is not null
+                    displayName: (node.user?.displayName || node.user?.username) ?? '',
+                    email: node.user?.email ?? '',
+                    user: node.user,
                 }}
             />
             <Timestamp date={node.createdAt} />