fix: change retrieval type (#404)

* change empty activity data from `null` to an empty array.
* ensure that proper message is displayed when the data is empty.

Author: Adrinlol

ui/packages/shared/pages/Instance/Info/Retrieval/RetrievalModal/index.tsx

@@ -14,8 +14,8 @@ export const RetrievalModal = ({
   isOpen: boolean
   onClose: () => void
   data: {
-    source: ActivityType[] | null
-    target: ActivityType[] | null
+    source: ActivityType[]
+    target: ActivityType[]
   }
 }) => {
   const stores = useStores()

ui/packages/shared/pages/Instance/Info/Retrieval/RetrievalTable/index.tsx

@@ -13,7 +13,7 @@ export const RetrievalTable = ({
   data,
   activity,
 }: {
-  data: ActivityType[] | null
+  data: ActivityType[]
   activity: string
 }) => {
   return (
@@ -24,7 +24,7 @@ export const RetrievalTable = ({
         </TableRow>
       </TableHead>
       <TableBody className={styles.tableBody}>
-        {data ? (
+        {data && data.length > 0 ? (
           data.map((item) => (
             <div>
               {Object.entries(item).map((val, index) => (
@@ -38,9 +38,7 @@ export const RetrievalTable = ({
           ))
         ) : (
           <TableRow className={styles.tableRow}>
-            <TableCell>
-              No activity on the {activity}
-            </TableCell>
+            <TableCell>No activity on the {activity}</TableCell>
           </TableRow>
         )}
       </TableBody>

ui/packages/shared/types/api/entities/instanceRetrieval.ts

@@ -14,7 +14,7 @@ export type InstanceRetrieval = {
   status: string
   currentJob?: string
   activity: {
-    source: ActivityType[] | null
-    target: ActivityType[] | null
-  }
-}
+    source: ActivityType[]
+    target: ActivityType[]
+  } | null
+}