🐛 Fix sorting on application inventory tables

Signed-off-by: Mike Turley <[email protected]>
konveyor · Sep 25, 2023 · 2410175 · 2410175
1 parent 70c029b
commit 2410175
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/client/src/app/pages/applications/applications-table-analyze/applications-table-analyze.tsx b/client/src/app/pages/applications/applications-table-analyze/applications-table-analyze.tsx
@@ -198,6 +198,12 @@ export const ApplicationsTableAnalyze: React.FC = () => {
     },
     sortableColumns: ["name", "description", "businessService", "tags"],
     initialSort: { columnKey: "name", direction: "asc" },
+    getSortValues: (app) => ({
+      name: app.name,
+      description: app.description || "",
+      businessService: app.businessService?.name || "",
+      tags: app.tags?.length || 0,
+    }),
     filterCategories: [
       {
         key: "name",
@@ -551,7 +557,7 @@ export const ApplicationsTableAnalyze: React.FC = () => {
             </ToolbarItem>
           </ToolbarContent>
         </Toolbar>
-        <Table {...tableProps} isExpandable aria-label="App analysis table">
+        <Table {...tableProps} aria-label="App analysis table">
           <Thead>
             <Tr>
               <TableHeaderContentWithControls {...tableControls}>
@@ -560,6 +566,7 @@ export const ApplicationsTableAnalyze: React.FC = () => {
                 <Th {...getThProps({ columnKey: "businessService" })} />
                 <Th {...getThProps({ columnKey: "analysis" })} />
                 <Th {...getThProps({ columnKey: "tags" })} />
+                <Th />
               </TableHeaderContentWithControls>
             </Tr>
           </Thead>

diff --git a/...rc/app/pages/applications/applications-table-assessment/applications-table-assessment.tsx b/...rc/app/pages/applications/applications-table-assessment/applications-table-assessment.tsx
@@ -244,6 +244,12 @@ export const ApplicationsTable: React.FC = () => {
     },
     sortableColumns: ["name", "description", "businessService", "tags"],
     initialSort: { columnKey: "name", direction: "asc" },
+    getSortValues: (app) => ({
+      name: app.name,
+      description: app.description || "",
+      businessService: app.businessService?.name || "",
+      tags: app.tags?.length || 0,
+    }),
     filterCategories: [
       {
         key: "name",
@@ -576,7 +582,7 @@ export const ApplicationsTable: React.FC = () => {
             </ToolbarItem>
           </ToolbarContent>
         </Toolbar>
-        <Table {...tableProps} isExpandable aria-label="App assessments table">
+        <Table {...tableProps} aria-label="App assessments table">
           <Thead>
             <Tr>
               <TableHeaderContentWithControls {...tableControls}>
@@ -586,6 +592,7 @@ export const ApplicationsTable: React.FC = () => {
                 <Th {...getThProps({ columnKey: "assessment" })} />
                 <Th {...getThProps({ columnKey: "review" })} />
                 <Th {...getThProps({ columnKey: "tags" })} />
+                <Th />
               </TableHeaderContentWithControls>
             </Tr>
           </Thead>