fix: sort cols

metrico · Jan 18, 2024 · 4a915fd · 4a915fd
1 parent a661b00
commit 4a915fd
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 12 deletions.
diff --git a/packages/main/plugins/Cardinality/TotalsPanel/CardinalityTotals.tsx b/packages/main/plugins/Cardinality/TotalsPanel/CardinalityTotals.tsx
@@ -19,20 +19,29 @@ export default function CardinalityTotals({ isLoading }) {
 
     const sortByProperty = useCallback(
         (column: string) => {
-            const numberCols = [];
+            const numberCols = [
+                "series_created",
+                "series_dropped",
+                "to_sec",
+                "from_sec",
+                "created_sec",
+            ];
+
+            const columnName = column.split(" ").join("_").toLocaleLowerCase();
 
             setTotals((prev) => {
                 let prevCols = [...prev];
-                if (numberCols.includes(column)) {
-                    return prevCols.sortColByNumber(column, sort);
+                if (numberCols.includes(columnName)) {
+                    return prevCols.sortColByNumber(columnName, sort);
                 }
-                return prevCols.sortColByString(column, sort);
+                return prevCols.sortColByString(columnName, sort);
             });
 
             setSort((prev) => (prev === "asc" ? "desc" : "asc"));
         },
         [totals]
     );
+
     return (
         <div className={cx(TotalRowStyle(theme))}>
             <div
@@ -50,7 +59,11 @@ export default function CardinalityTotals({ isLoading }) {
             <div className="table">
                 <div className="table-header">
                     {PROCESS_HEADERS?.map((header) => (
-                        <div key={header} className="cell">
+                        <div
+                            key={header}
+                            onClick={() => sortByProperty(header)}
+                            className="cell"
+                        >
                             {header}
                         </div>
                     ))}

diff --git a/packages/main/plugins/Cardinality/TotalsPanel/helper.tsx b/packages/main/plugins/Cardinality/TotalsPanel/helper.tsx
@@ -52,7 +52,6 @@ const TYPES = (type: string) => {
 };
 
 export function TypeRenderer({ type, children }) {
-    console.log(type);
 
     return <span style={{ color: TYPES(type) }}>{children}</span>;
 }
diff --git a/packages/main/plugins/Cardinality/api/mock.ts b/packages/main/plugins/Cardinality/api/mock.ts
@@ -7,9 +7,9 @@ export const totalsMock: MaintainanceItem[] = [
         query: "{a=\"b\"}",
         type: "delete",
         status: "running",
-        created_sec: 1704359839,
-        from_sec: 1704240000,
-        to_sec: 1704326400,
+        created_sec: 1704359849,
+        from_sec: 1704240010,
+        to_sec: 1704326420,
         series_dropped: 150,
         series_created: 0,
         logs: [
@@ -39,9 +39,9 @@ export const totalsMock: MaintainanceItem[] = [
         query: "{__name=\"server\",type=\"clickhouse\",level=\"info\"}",
         type: "delete",
         status: "running",
-        created_sec: 1704359839,
-        from_sec: 1704240000,
-        to_sec: 1704326400,
+        created_sec: 1704359859,
+        from_sec: 1704240020,
+        to_sec: 1704326415,
         series_dropped: 150,
         series_created: 0,
         logs: [