fix: dont show headline if we dont have a thumbprint / certificateDet…

…ails (#18435)
wireapp · Dec 5, 2024 · 50f9bce · 50f9bce
1 parent 2adab05
commit 50f9bce
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/...nt/panels/preferences/DevicesPreferences/components/MLSDeviceDetails/MLSDeviceDetails.tsx b/...nt/panels/preferences/DevicesPreferences/components/MLSDeviceDetails/MLSDeviceDetails.tsx
@@ -40,11 +40,17 @@ export const MLSDeviceDetails = ({isCurrentDevice, identity, isSelfUser = false}
 
   const certificateState = identity?.status ?? MLSStatuses.NOT_ACTIVATED;
   const isE2EIEnabled = E2EIHandler.getInstance().isE2EIEnabled();
+  const showE2EICertificateDetails =
+    isE2EIEnabled && (isSelfUser || (!isSelfUser && certificateState !== MLSStatuses.NOT_ACTIVATED));
 
   if (!isSelfUser && certificateState === MLSStatuses.NOT_ACTIVATED) {
     return null;
   }
 
+  if (!showE2EICertificateDetails && !identity?.thumbprint) {
+    return null;
+  }
+
   return (
     <div css={styles.wrapper}>
       <h4 className="paragraph-body-3">{t('mlsSignature', {signature: MLSPublicKeys.ED25519.toUpperCase()})}</h4>
@@ -59,9 +65,7 @@ export const MLSDeviceDetails = ({isCurrentDevice, identity, isSelfUser = false}
         </>
       )}
 
-      {isE2EIEnabled && (isSelfUser || (!isSelfUser && certificateState !== MLSStatuses.NOT_ACTIVATED)) && (
-        <E2EICertificateDetails identity={identity} isCurrentDevice={isCurrentDevice} />
-      )}
+      {showE2EICertificateDetails && <E2EICertificateDetails identity={identity} isCurrentDevice={isCurrentDevice} />}
     </div>
   );
 };