diff --git a/Client/src/Components/BasicTable/index.jsx b/Client/src/Components/BasicTable/index.jsx
index 7457b7abf..705872033 100644
--- a/Client/src/Components/BasicTable/index.jsx
+++ b/Client/src/Components/BasicTable/index.jsx
@@ -17,45 +17,52 @@ import PropTypes from "prop-types";
*
* @component
* @param {Object} props - Component props.
- * @param {Array} props.headers - Array of header titles for the table.
- * @param {Array} props.rowItems - Array of row data objects for the table.
- * @param {boolean} props.paginated - Flag to enable pagination.
- * @param {function} props.handleClick - Function to handle click on row.
+ * @param {Object} props.data - Data for the table including columns and rows.
+ * @param {Array} props.data.cols - Array of objects for column headers.
+ * @param {number} props.data.cols[].id - Unique identifier for the column.
+ * @param {string} props.data.cols[].name - Name of the column to display as header.
+ * @param {Array} props.data.rows - Array of row objects.
+ * @param {number} props.data.rows[].id - Unique identifier for the row.
+ * @param {Array} props.data.rows[].data - Array of cell data objects for the row.
+ * @param {number} props.data.rows[].data[].id - Unique identifier for the cell.
+ * @param {JSX.Element} props.data.rows[].data[].data - The content to display in the cell.
+ * @param {functio} props.data.rows.data.handleClick - Function to call when the row is clicked.
+ * @param {boolean} [props.paginated=false] - Flag to enable pagination.
*
* @example
+ * const data = {
+ * cols: [
+ * { id: 1, name: "First Col" },
+ * { id: 2, name: "Second Col" },
+ * { id: 3, name: "Third Col" },
+ * { id: 4, name: "Fourth Col" },
+ * ],
+ * rows: [
+ * {
+ * id: 1,
+ * data: [
+ * { id: 1, data:
Data for Row 1 Col 1
},
+ * { id: 2, data: Data for Row 1 Col 2
},
+ * { id: 3, data: Data for Row 1 Col 3
},
+ * { id: 4, data: Data for Row 1 Col 4
},
+ * ],
+ * },
+ * {
+ * id: 2,
+ * data: [
+ * { id: 5, data: Data for Row 2 Col 1
},
+ * { id: 6, data: Data for Row 2 Col 2
},
+ * { id: 7, data: Data for Row 2 Col 3
},
+ * { id: 8, data: Data for Row 2 Col 4
},
+ * ],
+ * },
+ * ],
+ * };
*
- * Headers is an array of objets that have a unique ID and a name
- * Rows is an array of objects that have a unique ID and some data to display
- * Row data can be any valid React node
- * Rows are split into groups according to the number of headers, so if you have 3 headers then
- * Rows will go under the index % 3 column.
- * ie: Row 0 -> col 0 % 3 = 0
- * ie: Row 1 -> col 1 % 3 = 1
- * ie: Row 2 -> col 2 % 3 = 2
- * ie: Row 3 -> col 3 % 3 = 0
- *
- * Thus items are put in their correct columns
- *
- *
- * const headers = [
- * { id: 1, name: "ID" },
- * { id: 2, name: "Name" },
- * { id: 3, name: "Age" },
- * ];
- * const rows = [
- * { id: 1, data: 999 },
- * { id: 2, data: Alex Holliday
},
- * { id: 3, data: 5
},
- * // More rows...
- * ];
- *
- * ROW CLICK HANDLING
- * If you want to set a listener for a row click, you can add a handleClick function to FIRST ITEM IN EACH ROW
- *
- *
+ *
*/
-const BasicTable = ({ headers, rowItems, paginated }) => {
+const BasicTable = ({ data, paginated }) => {
// Add headers to props validation
const [page, setPage] = useState(0);
@@ -70,19 +77,13 @@ const BasicTable = ({ headers, rowItems, paginated }) => {
setPage(0); // Reset to first page after changing rows per page
};
- const numOfCols = headers.length;
-
- const groupedRowItems = rowItems.reduce((acc, item, idx) => {
- const rowIndex = Math.floor((item.id - 1) / numOfCols);
- if (!acc[rowIndex]) acc[rowIndex] = { items: [], idx };
- acc[rowIndex].items.push({ item });
- return acc;
- }, {});
-
- const paginatedRowsKeys = Object.keys(groupedRowItems).slice(
- page * rowsPerPage,
- page * rowsPerPage + rowsPerPage
- );
+ let paginatedData = [];
+ if (paginated) {
+ paginatedData = data.rows.slice(
+ page * rowsPerPage,
+ page * rowsPerPage + rowsPerPage
+ );
+ }
return (
<>
@@ -90,56 +91,52 @@ const BasicTable = ({ headers, rowItems, paginated }) => {
- {headers.map((header) => {
- return {header.name};
- })}
+ {data.cols.map((col) => (
+ {col.name}
+ ))}
{paginated === false &&
- Object.keys(groupedRowItems).map((key) => {
+ data.rows.map((row) => {
return (
{
- groupedRowItems[key].items[0].item.handleClick
- ? groupedRowItems[key].items[0].item.handleClick()
- : null;
- }}
- key={groupedRowItems[key].idx}
+ sx={{ cursor: row.handleClick ? "pointer" : "default" }}
+ key={row.id}
+ onClick={row.handleClick ? row.handleClick : null}
>
- {groupedRowItems[key].items.map(({ item }) => {
- return {item.data};
+ {row.data.map((cell) => {
+ return {cell.data};
})}
);
})}
{paginated === true &&
- paginatedRowsKeys.map((key) => (
- {
- groupedRowItems[key].items[0].item.handleClick
- ? groupedRowItems[key].items[0].item.handleClick()
- : null;
- }}
- key={groupedRowItems[key].idx}
- >
- {groupedRowItems[key].items.map(({ item }) => (
- {item.data}
- ))}
-
- ))}
+ paginatedData.map((row) => {
+ return (
+
+ {row.data.map((cell) => {
+ return {cell.data};
+ })}
+
+ );
+ })}
{paginated === true && (
)}
>
@@ -147,20 +144,8 @@ const BasicTable = ({ headers, rowItems, paginated }) => {
};
BasicTable.propTypes = {
- headers: PropTypes.arrayOf(
- PropTypes.shape({
- id: PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired,
- name: PropTypes.string.isRequired,
- })
- ).isRequired,
- rowItems: PropTypes.arrayOf(
- PropTypes.shape({
- id: PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired,
- data: PropTypes.node,
- })
- ).isRequired,
+ data: PropTypes.object.isRequired,
paginated: PropTypes.bool,
- handleClick: PropTypes.func,
};
export default BasicTable;
diff --git a/Client/src/Components/MonitorTable/index.jsx b/Client/src/Components/MonitorTable/index.jsx
index f26b44a78..35b7a0fca 100644
--- a/Client/src/Components/MonitorTable/index.jsx
+++ b/Client/src/Components/MonitorTable/index.jsx
@@ -83,14 +83,17 @@ const Status = ({ params }) => {
const MonitorTable = ({ monitors = [] }) => {
const navigate = useNavigate();
- const headers = [
- { id: 1, name: "Host" },
- { id: 2, name: "Status" },
- { id: 3, name: "Response Time" },
- { id: 4, name: "Actions" },
- ];
- const data = [];
- monitors.forEach((monitor) => {
+ const data = {
+ cols: [
+ { id: 1, name: "Host" },
+ { id: 2, name: "Status" },
+ { id: 3, name: "Response Time" },
+ { id: 4, name: "Actions" },
+ ],
+ rows: [],
+ };
+
+ data.rows = monitors.map((monitor, idx) => {
const params = {
url: monitor.url,
title: monitor.name,
@@ -109,37 +112,20 @@ const MonitorTable = ({ monitors = [] }) => {
? "var(--env-var-color-17)"
: "var(--env-var-color-19)",
};
- data.push({
- id: data.length + 1,
- data: ,
- handleClick: () => {
- navigate(`/monitors/${monitor._id}`);
- },
- });
- data.push({
- id: data.length + 1,
- data: ,
- });
- data.push({
- id: data.length + 1,
- data: ,
- handleClick: () => {
- console.log(monitor._id);
- },
- });
- data.push({
- id: data.length + 1,
- data: ,
- });
- data.push({
- id: data.length + 1,
- data: ,
- });
- data.push({ id: data.length + 1, itemId: monitor._id, data: "TODO" });
+ return {
+ id: monitor._id,
+ handleClick: () => navigate(`/monitors/${monitor._id}`),
+ data: [
+ { id: idx, data: },
+ { id: idx + 1, data: },
+ { id: idx + 2, data: },
+ { id: idx + 3, data: "TODO" },
+ ],
+ };
});
- return ;
+ return ;
};
MonitorTable.propTypes = {
diff --git a/Client/src/Pages/Details/index.jsx b/Client/src/Pages/Details/index.jsx
index 488b2b6ab..201a32524 100644
--- a/Client/src/Pages/Details/index.jsx
+++ b/Client/src/Pages/Details/index.jsx
@@ -27,7 +27,6 @@ const DetailsPage = () => {
const theme = useTheme();
- // Example monitor data (replace with actual data or fetch from API)
/**
* Function to calculate uptime duration based on the most recent check.
* @param {Array} checks Array of check objects.