cleaned up code

Author: jaenixlee

src/components/Images/Images.tsx

@@ -12,8 +12,9 @@ import globalStyles from '../global.module.scss';
  * @description | Provides ability to pull images from DockerHub image repository, run images, and remove images
  **/
 
+// eslint-disable-next-line react/prop-types
 const Images = ({ imagesList: initialImagesList }): JSX.Element => {
-  // const imagesList = useAppSelector((state) => state.images.imagesList);
+  // imagesList for testing purposes only
   const reduxImagesList = useAppSelector((state) => state.images.imagesList);
   const imagesList = initialImagesList ? [initialImagesList] : reduxImagesList;
   const [repo, setRepo] = useState('');

src/components/Metrics/Metrics.tsx

@@ -4,11 +4,11 @@ import useHelper from '../../helpers/commands';
 
 const Metrics = (): JSX.Element => {
   const { getKey, getUid } = useHelper();
-  const [kubernetesOrNah, setFrame] = useState(1);
+  const [toggleKubernetes, setToggleKubernetes] = useState(1);
   const [apiKey, setApiKey] = useState('');
   const [uidKey, setUidKey] = useState('');
   const [currentPage, setCurrentPage] = useState('Node Exporter / Nodes');
-  const button = kubernetesOrNah === 1 ? 'Containers' : 'Kubernetes Cluster';
+  const button = toggleKubernetes === 1 ? 'Containers' : 'Kubernetes Cluster';
 
   useEffect(() => {
     /** 
@@ -31,7 +31,7 @@ const Metrics = (): JSX.Element => {
   }, []);
 
   const handleToggle = () => {
-    setFrame((prevFrame) => (prevFrame === 1 ? 2 : 1));
+    setToggleKubernetes((prevFrame) => (prevFrame === 1 ? 2 : 1));
   };
 
   /** 
@@ -57,7 +57,7 @@ const Metrics = (): JSX.Element => {
       <>{button}</>
       <label className={styles.toggle} htmlFor="switch" />
 
-      {kubernetesOrNah === 1 ? (
+      {toggleKubernetes === 1 ? (
         <div>
           <iframe
             className={styles.metrics}

src/helpers/dispatch.tsx

@@ -15,16 +15,6 @@ import {
   refreshStoppedContainer,
 } from '../reducers/containerReducer';
 
-import {
-  buildAxis,
-  buildMemory,
-  buildCpu,
-  buildWrittenIO,
-  buildReadIO,
-  buildReceivedIO,
-  buildTransmittedIO,
-} from '../reducers/graphReducer';
-
 import { refreshImages } from '../reducers/imageReducer';
 
 import { getLogs } from '../reducers/logReducer';
@@ -108,28 +98,6 @@ const useSurvey = () => {
       composeDown(data: any) {
         dispatch(composeDown(data));
       },
-      // Dispatch functions used in LineChartDisplay.tsx
-      buildAxis(data: any) {
-        dispatch(buildAxis(data));
-      },
-      buildMemory(data: any) {
-        dispatch(buildMemory(data));
-      },
-      buildCpu(data: any) {
-        dispatch(buildCpu(data));
-      },
-      buildWrittenIO(data: any) {
-        dispatch(buildWrittenIO(data));
-      },
-      buildReadIO(data: any) {
-        dispatch(buildReadIO(data));
-      },
-      buildReceivedIO(data: any) {
-        dispatch(buildReceivedIO(data));
-      },
-      buildTransmittedIO(data: any) {
-        dispatch(buildTransmittedIO(data));
-      },
       // Dispatch functions used in ProcessLogsTable.tsx
       getContainerLogsDispatcher(data: ContainerLogsType) {
         dispatch(getLogs(data));

src/reducers/alertReducer.ts

@@ -42,7 +42,6 @@ const alertSlice = createSlice({
 
 export const { setAlert, setPrompt } = alertSlice.actions;
 
-// let timeoutId = null;
 let timeoutId: ReturnType<typeof setTimeout> | null = null;
 
 export const createAlert = (

src/reducers/composeReducer.ts

@@ -2,7 +2,6 @@ import { PayloadAction, createSlice } from '@reduxjs/toolkit';
 import { ContainerStateType } from '../../types';
 
 const initialState: ContainerStateType = {
-  // ? why is runningList & stoppedList here
   runningList: [],
   stoppedList: [],
   networkList: [],
@@ -20,10 +19,8 @@ export const composeSlice = createSlice({
       const currentState: any = state.composeStack;
 
       const composeStackUpdater = (
-        // ? are the two forEaches redundant?
         // current state is first array
         firstArray: [],
-        // action payload is second array (new stack?)
         secondArray: [],
         outputArray = []
       ) => {

src/reducers/graphReducer.ts

@@ -10,16 +10,11 @@ const initialState: SessionStateType = {
   _id: '',
   username: '',
   password: '',
-  email: '',
-  phone: '',
-  role: '',
-  contact_pref: '',
   mem_threshold: '',
   cpu_threshold: '',
   container_stops: '',
   token: '',
   isLoggedIn: false,
-  // userList: [],
 };
 
 export const sessionSlice = createSlice({

src/reducers/sessionReducer.ts

@@ -2,7 +2,6 @@ import { configureStore } from '@reduxjs/toolkit';
 import alertReducer from './reducers/alertReducer';
 import composeReducer from './reducers/composeReducer';
 import containerReducer from './reducers/containerReducer';
-import graphReducer from './reducers/graphReducer';
 import imageReducer from './reducers/imageReducer';
 import logReducer from './reducers/logReducer';
 import userReducer from './reducers/userReducer';
@@ -13,7 +12,6 @@ const store = configureStore({
   reducer: {
     containers: containerReducer,
     images: imageReducer,
-    graphs: graphReducer,
     composes: composeReducer,
     sessions: sessionReducer,
     volumes: volumeReducer,

src/store.ts

@@ -24,10 +24,6 @@ export interface UserInfo extends User {
   // removed password property on userInfo as it's not being used.
   // changed id from number type to string type so see if that breaks anything
   _id: string;
-  email: string;
-  phone: string;
-  role: string;
-  contact_pref: string;
   // changed memthreshold from string to string to align with sessionState in sessions reducer. see if it broke something
   mem_threshold: string;
   // changed cpu threshold from string to string let's see what happens