diff --git a/Client/src/Components/TabPanels/Account/ProfilePanel.jsx b/Client/src/Components/TabPanels/Account/ProfilePanel.jsx index 355b09ca6..398c10c54 100644 --- a/Client/src/Components/TabPanels/Account/ProfilePanel.jsx +++ b/Client/src/Components/TabPanels/Account/ProfilePanel.jsx @@ -19,6 +19,7 @@ import ProgressUpload from "../../ProgressBars"; import { formatBytes } from "../../../Utils/fileUtils"; import { clearUptimeMonitorState } from "../../../Features/UptimeMonitors/uptimeMonitorsSlice"; import { createToast } from "../../../Utils/toastUtils"; +import { logger } from "../../../Utils/Logger"; /** * ProfilePanel component displays a form for editing user profile information @@ -256,7 +257,7 @@ const ProfilePanel = () => { placeholder="Enter your email" autoComplete="email" // TODO - add onChange - onChange={() => console.log("Disabled.")} + onChange={() => logger.warn("disabled")} // error={errors[idToName["edit-email"]]} disabled={true} /> diff --git a/Client/src/HOC/withAdminCheck.jsx b/Client/src/HOC/withAdminCheck.jsx index 55e94d4db..30a767f36 100644 --- a/Client/src/HOC/withAdminCheck.jsx +++ b/Client/src/HOC/withAdminCheck.jsx @@ -1,5 +1,7 @@ import React, { useEffect } from "react"; import { useNavigate } from "react-router-dom"; + +import { logger } from "../Utils/Logger"; import { networkService } from "../main"; const withAdminCheck = (WrappedComponent) => { @@ -15,7 +17,7 @@ const withAdminCheck = (WrappedComponent) => { } }) .catch((error) => { - console.log(error); + logger.error(error); }); }, [navigate]); return ; diff --git a/Client/src/Pages/Auth/Login.jsx b/Client/src/Pages/Auth/Login.jsx index 2776866dc..79e8d7e12 100644 --- a/Client/src/Pages/Auth/Login.jsx +++ b/Client/src/Pages/Auth/Login.jsx @@ -14,7 +14,7 @@ import Logo from "../../assets/icons/bwu-icon.svg?react"; import Mail from "../../assets/icons/mail.svg?react"; import ArrowBackRoundedIcon from "@mui/icons-material/ArrowBackRounded"; import PropTypes from "prop-types"; - +import { logger } from "../../Utils/Logger"; import "./index.css"; /** @@ -284,7 +284,7 @@ const Login = () => { } }) .catch((error) => { - console.log(error); + logger.error(error); }); }, [authToken, navigate]); diff --git a/Client/src/Pages/Auth/Register/Register.jsx b/Client/src/Pages/Auth/Register/Register.jsx index b7dd70dc9..225fb16f0 100644 --- a/Client/src/Pages/Auth/Register/Register.jsx +++ b/Client/src/Pages/Auth/Register/Register.jsx @@ -17,6 +17,7 @@ import Button from "../../../Components/Button"; import Field from "../../../Components/Inputs/Field"; import { networkService } from "../../../main"; import "../index.css"; +import { logger } from "../../../Utils/Logger"; /** * Displays the initial landing page. @@ -399,10 +400,9 @@ const Register = ({ isAdmin }) => { try { const res = await networkService.verifyInvitationToken(token); const { role, email } = res.data.data; - console.log(role); setForm({ ...form, email, role }); } catch (error) { - console.log(error); + logger.error(error); } } }; diff --git a/Client/src/Pages/Incidents/IncidentTable/index.jsx b/Client/src/Pages/Incidents/IncidentTable/index.jsx index e476eac1f..705daa70c 100644 --- a/Client/src/Pages/Incidents/IncidentTable/index.jsx +++ b/Client/src/Pages/Incidents/IncidentTable/index.jsx @@ -20,6 +20,7 @@ import { useState, useEffect } from "react"; import { useSelector } from "react-redux"; import { networkService } from "../../../main"; import { StatusLabel } from "../../../Components/Label"; +import { logger } from "../../../Utils/Logger"; const IncidentTable = ({ monitors, selectedMonitor, filter }) => { const { authToken, user } = useSelector((state) => state.auth); @@ -70,7 +71,7 @@ const IncidentTable = ({ monitors, selectedMonitor, filter }) => { setChecks(res.data.data.checks); setChecksCount(res.data.data.checksCount); } catch (error) { - console.log(error); + logger.error(error); } }; fetchPage(); @@ -85,7 +86,6 @@ const IncidentTable = ({ monitors, selectedMonitor, filter }) => { ]); const handlePageChange = (_, newPage) => { - console.log(newPage); setPaginationController({ ...paginationController, page: newPage - 1, // 0-indexed diff --git a/Client/src/Pages/Monitors/Configure/index.jsx b/Client/src/Pages/Monitors/Configure/index.jsx index c873482a9..86ea62960 100644 --- a/Client/src/Pages/Monitors/Configure/index.jsx +++ b/Client/src/Pages/Monitors/Configure/index.jsx @@ -19,6 +19,7 @@ import { import Checkbox from "../../../Components/Inputs/Checkbox"; import Breadcrumbs from "../../../Components/Breadcrumbs"; import "./index.css"; +import { logger } from "../../../Utils/Logger"; /** * Parses a URL string and returns a URL object. @@ -108,7 +109,7 @@ const Configure = () => { useEffect(() => { const data = monitors.find((monitor) => monitor._id === monitorId); if (!data) { - console.error("Error fetching monitor of id: " + monitorId); + logger.error("Error fetching monitor of id: " + monitorId); navigate("/not-found"); } setMonitor({ @@ -337,7 +338,7 @@ const Configure = () => { label="Notify via SMS (coming soon)" isChecked={false} value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} isDisabled={true} /> { label="Also notify via email to multiple addresses (coming soon)" isChecked={false} value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} isDisabled={true} /> {monitor?.notifications?.some( @@ -368,7 +369,7 @@ const Configure = () => { type="text" placeholder="name@gmail.com" value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} /> You can separate multiple emails with a comma diff --git a/Client/src/Pages/Monitors/CreateMonitor/index.jsx b/Client/src/Pages/Monitors/CreateMonitor/index.jsx index d0eafb476..442558f8e 100644 --- a/Client/src/Pages/Monitors/CreateMonitor/index.jsx +++ b/Client/src/Pages/Monitors/CreateMonitor/index.jsx @@ -13,6 +13,7 @@ import Select from "../../../Components/Inputs/Select"; import Checkbox from "../../../Components/Inputs/Checkbox"; import { createToast } from "../../../Utils/toastUtils"; import Breadcrumbs from "../../../Components/Breadcrumbs"; +import { logger } from "../../../Utils/Logger"; const CreateMonitor = () => { const MS_PER_MINUTE = 60000; @@ -267,7 +268,7 @@ const CreateMonitor = () => { label="Notify via SMS (coming soon)" isChecked={false} value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} isDisabled={true} /> { label="Also notify via email to multiple addresses (coming soon)" isChecked={false} value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} isDisabled={true} /> {monitor.notifications.some( @@ -296,7 +297,7 @@ const CreateMonitor = () => { type="text" placeholder="name@gmail.com" value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} /> You can separate multiple emails with a comma diff --git a/Client/src/Pages/Monitors/Details/PaginationTable/index.jsx b/Client/src/Pages/Monitors/Details/PaginationTable/index.jsx index 28795dff5..da4a0cb1e 100644 --- a/Client/src/Pages/Monitors/Details/PaginationTable/index.jsx +++ b/Client/src/Pages/Monitors/Details/PaginationTable/index.jsx @@ -17,6 +17,7 @@ import { networkService } from "../../../../main"; import { StatusLabel } from "../../../../Components/Label"; import ArrowBackRoundedIcon from "@mui/icons-material/ArrowBackRounded"; import ArrowForwardRoundedIcon from "@mui/icons-material/ArrowForwardRounded"; +import { logger } from "../../../../Utils/Logger"; const PaginationTable = ({ monitorId, dateRange }) => { const { authToken } = useSelector((state) => state.auth); @@ -50,7 +51,7 @@ const PaginationTable = ({ monitorId, dateRange }) => { setChecks(res.data.data.checks); setChecksCount(res.data.data.checksCount); } catch (error) { - console.log(error); + logger.error(error); } }; fetchPage(); diff --git a/Client/src/Pages/Monitors/Details/index.jsx b/Client/src/Pages/Monitors/Details/index.jsx index dad80f4be..85f3a0bb8 100644 --- a/Client/src/Pages/Monitors/Details/index.jsx +++ b/Client/src/Pages/Monitors/Details/index.jsx @@ -18,6 +18,7 @@ import { import "./index.css"; import MonitorDetails60MinChart from "../../../Components/Charts/MonitorDetails60MinChart"; import Breadcrumbs from "../../../Components/Breadcrumbs"; +import { logger } from "../../../Utils/Logger"; const StatBox = ({ title, value }) => { return ( @@ -132,8 +133,7 @@ const DetailsPage = () => { ); setMonitor(res.data.data); } catch (error) { - console.log(error); - console.error("Error fetching monitor of id: " + monitorId); + logger.error(error); navigate("/not-found"); } }, [authToken, monitorId, navigate, dateRange]); diff --git a/Client/src/Pages/Monitors/index.jsx b/Client/src/Pages/Monitors/index.jsx index 1b541c72b..c33cf16dc 100644 --- a/Client/src/Pages/Monitors/index.jsx +++ b/Client/src/Pages/Monitors/index.jsx @@ -28,6 +28,7 @@ import { import Settings from "../../assets/icons/settings-bold.svg?react"; import PropTypes from "prop-types"; +import { logger } from "../../Utils/Logger"; const ActionsMenu = ({ monitor }) => { const [anchorEl, setAnchorEl] = useState(null); @@ -36,7 +37,6 @@ const ActionsMenu = ({ monitor }) => { const dispatch = useDispatch(); const theme = useTheme(); const authState = useSelector((state) => state.auth); - const handleRemove = async (event) => { event.preventDefault(); event.stopPropagation(); diff --git a/Client/src/Pages/NotFound/index.jsx b/Client/src/Pages/NotFound/index.jsx index 68120b0a3..ef4b37839 100644 --- a/Client/src/Pages/NotFound/index.jsx +++ b/Client/src/Pages/NotFound/index.jsx @@ -33,7 +33,6 @@ const DefaultValue = { */ const NotFound = ({ title = DefaultValue.title, desc = DefaultValue.desc }) => { const navigate = useNavigate(); - console.log("NOT FOUND"); return ( diff --git a/Client/src/Pages/PageSpeed/Configure/index.jsx b/Client/src/Pages/PageSpeed/Configure/index.jsx index 0dff8d7eb..36a136165 100644 --- a/Client/src/Pages/PageSpeed/Configure/index.jsx +++ b/Client/src/Pages/PageSpeed/Configure/index.jsx @@ -18,6 +18,7 @@ import PauseCircleOutlineIcon from "@mui/icons-material/PauseCircleOutline"; import GreenCheck from "../../../assets/icons/checkbox-green.svg?react"; import RedCheck from "../../../assets/icons/checkbox-red.svg?react"; import Breadcrumbs from "../../../Components/Breadcrumbs"; +import { logger } from "../../../Utils/Logger"; import "./index.css"; @@ -92,7 +93,7 @@ const PageSpeedConfigure = () => { useEffect(() => { const data = monitors.find((monitor) => monitor._id === monitorId); if (!data) { - console.error("Error fetching pagespeed monitor of id: " + monitorId); + logger.error("Error fetching pagespeed monitor of id: " + monitorId); navigate("/not-found"); } setMonitor({ @@ -278,7 +279,7 @@ const PageSpeedConfigure = () => { id="notify-emails-list" placeholder="notifications@gmail.com" value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} error="" /> diff --git a/Client/src/Pages/PageSpeed/CreatePageSpeed/index.jsx b/Client/src/Pages/PageSpeed/CreatePageSpeed/index.jsx index 6b9e6dbde..c707db319 100644 --- a/Client/src/Pages/PageSpeed/CreatePageSpeed/index.jsx +++ b/Client/src/Pages/PageSpeed/CreatePageSpeed/index.jsx @@ -12,6 +12,7 @@ import { createToast } from "../../../Utils/toastUtils"; import { createPageSpeed } from "../../../Features/PageSpeedMonitor/pageSpeedMonitorSlice"; import Breadcrumbs from "../../../Components/Breadcrumbs"; import "./index.css"; +import { logger } from "../../../Utils/Logger"; const CreatePageSpeed = () => { const theme = useTheme(); @@ -175,7 +176,7 @@ const CreatePageSpeed = () => { id="notify-emails-list" placeholder="notifications@gmail.com" value="" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} error="" /> diff --git a/Client/src/Pages/PageSpeed/Details/index.jsx b/Client/src/Pages/PageSpeed/Details/index.jsx index 4155170b6..adcc51a3c 100644 --- a/Client/src/Pages/PageSpeed/Details/index.jsx +++ b/Client/src/Pages/PageSpeed/Details/index.jsx @@ -21,6 +21,7 @@ import PageSpeedLineChart from "../../../Components/Charts/PagespeedLineChart"; import Breadcrumbs from "../../../Components/Breadcrumbs"; import "./index.css"; import PropTypes from "prop-types"; +import { logger } from "../../../Utils/Logger"; const StatBox = ({ icon, title, value }) => { const theme = useTheme(); @@ -210,7 +211,7 @@ const PageSpeedDetails = () => { setMonitor(res?.data?.data ?? {}); setAudits(res?.data?.data?.checks?.[0]?.audits ?? []); } catch (error) { - console.log(error); + logger.error(logger); navigate("/not-found"); } }; diff --git a/Client/src/Pages/Settings/index.jsx b/Client/src/Pages/Settings/index.jsx index 3ee029675..2b6f40682 100644 --- a/Client/src/Pages/Settings/index.jsx +++ b/Client/src/Pages/Settings/index.jsx @@ -4,7 +4,7 @@ import Button from "../../Components/Button"; import Field from "../../Components/Inputs/Field"; import Link from "../../Components/Link"; import Select from "../../Components/Inputs/Select"; - +import { logger } from "../../Utils/Logger"; import "./index.css"; const Settings = () => { @@ -40,14 +40,14 @@ const Settings = () => { id="display-timezone" label="Display timezone" value="est" - onChange={() => console.log("disabled")} + onChange={() => logger.warn("disabled")} items={[{ _id: "est", name: "America / Toronto" }]} />