From 0c8719057f1fcfd9869be9d07f8a1efac5388934 Mon Sep 17 00:00:00 2001
From: Hugh Nimmo-Smith <hughns@matrix.org>
Date: Fri, 1 Sep 2023 16:52:01 +0100
Subject: [PATCH] Only show client details if available

---
 .../SessionDetail/CompatSessionDetail.tsx     | 21 +++++++++++--------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/frontend/src/components/SessionDetail/CompatSessionDetail.tsx b/frontend/src/components/SessionDetail/CompatSessionDetail.tsx
index 40a43a0b0..c11d3dc5d 100644
--- a/frontend/src/components/SessionDetail/CompatSessionDetail.tsx
+++ b/frontend/src/components/SessionDetail/CompatSessionDetail.tsx
@@ -54,28 +54,31 @@ const CompatSessionDetail: React.FC<Props> = ({ session }) => {
     ...finishedAt,
   ];
 
-  const clientName = data.ssoLogin?.redirectUri
-    ? simplifyUrl(data.ssoLogin.redirectUri)
-    : undefined;
+  const clientDetails: { label: string; value: string | JSX.Element }[] = [];
 
-  const clientDetails = [
-    { label: "Name", value: clientName },
-    {
+  if (data.ssoLogin?.redirectUri) {
+    clientDetails.push({
+      label: "Name",
+      value: simplifyUrl(data.ssoLogin.redirectUri),
+    });
+    clientDetails.push({
       label: "Uri",
       value: (
         <a target="_blank" href={data.ssoLogin?.redirectUri}>
           {data.ssoLogin?.redirectUri}
         </a>
       ),
-    },
-  ];
+    });
+  }
 
   return (
     <div>
       <BlockList>
         <H3>{data.deviceId || data.id}</H3>
         <SessionDetails title="Session" details={sessionDetails} />
-        <SessionDetails title="Client" details={clientDetails} />
+        {clientDetails.length > 0 ? (
+          <SessionDetails title="Client" details={clientDetails} />
+        ) : null}
         {!data.finishedAt && (
           <Button
             kind="destructive"