From aaba117a43f685f388416e3ca2d1e89cf85a7af4 Mon Sep 17 00:00:00 2001
From: yaacov <kobi.zamir@gmail.com>
Date: Wed, 29 May 2024 12:49:04 +0300
Subject: [PATCH] Hide empty conditions sections

Signed-off-by: yaacov <kobi.zamir@gmail.com>
---
 .../MigrationVirtualMachinesRowExtended.tsx   | 60 ++++++++++---------
 1 file changed, 32 insertions(+), 28 deletions(-)

diff --git a/packages/forklift-console-plugin/src/modules/Plans/views/details/tabs/VirtualMachines/Migration/MigrationVirtualMachinesRowExtended.tsx b/packages/forklift-console-plugin/src/modules/Plans/views/details/tabs/VirtualMachines/Migration/MigrationVirtualMachinesRowExtended.tsx
index b12e82db1..ab63c088c 100644
--- a/packages/forklift-console-plugin/src/modules/Plans/views/details/tabs/VirtualMachines/Migration/MigrationVirtualMachinesRowExtended.tsx
+++ b/packages/forklift-console-plugin/src/modules/Plans/views/details/tabs/VirtualMachines/Migration/MigrationVirtualMachinesRowExtended.tsx
@@ -156,34 +156,38 @@ export const MigrationVirtualMachinesRowExtended: React.FC<RowProps<VMData>> = (
         </>
       )}
 
-      <SectionHeading
-        text={'Conditions'}
-        className="forklift-page-plan-details-vm-status__section-header"
-      />
-      <TableComposable aria-label="Expandable table" variant="compact">
-        <Thead>
-          <Tr>
-            <Th width={10}>{t('Type')}</Th>
-            <Th width={10}>{t('Status')}</Th>
-            <Th width={20}>{t('Updated')}</Th>
-            <Th width={10}>{t('Reason')}</Th>
-            <Th> {t('Message')}</Th>
-          </Tr>
-        </Thead>
-        <Tbody>
-          {(conditions || []).map((condition) => (
-            <Tr key={condition.type}>
-              <Td>{condition.type}</Td>
-              <Td>{getStatusLabel(condition.status)}</Td>
-              <Td>
-                <Timestamp timestamp={condition.lastTransitionTime} />
-              </Td>
-              <Td>{condition.reason}</Td>
-              <Td modifier="truncate">{condition?.message || '-'}</Td>
-            </Tr>
-          ))}
-        </Tbody>
-      </TableComposable>
+      {(conditions || []).length > 0 && (
+        <>
+          <SectionHeading
+            text={'Conditions'}
+            className="forklift-page-plan-details-vm-status__section-header"
+          />
+          <TableComposable aria-label="Expandable table" variant="compact">
+            <Thead>
+              <Tr>
+                <Th width={10}>{t('Type')}</Th>
+                <Th width={10}>{t('Status')}</Th>
+                <Th width={20}>{t('Updated')}</Th>
+                <Th width={10}>{t('Reason')}</Th>
+                <Th> {t('Message')}</Th>
+              </Tr>
+            </Thead>
+            <Tbody>
+              {(conditions || []).map((condition) => (
+                <Tr key={condition.type}>
+                  <Td>{condition.type}</Td>
+                  <Td>{getStatusLabel(condition.status)}</Td>
+                  <Td>
+                    <Timestamp timestamp={condition.lastTransitionTime} />
+                  </Td>
+                  <Td>{condition.reason}</Td>
+                  <Td modifier="truncate">{condition?.message || '-'}</Td>
+                </Tr>
+              ))}
+            </Tbody>
+          </TableComposable>
+        </>
+      )}
 
       <SectionHeading
         text={'Pipeline'}