Merge pull request #77 from DerekQuoc/comments

Comments

Author: mjiang108

app/charts/ResponseCodesChart.tsx

@@ -50,8 +50,6 @@ const ResponseCodesChart: React.FC = React.memo(() => {
     // Record each status code frequencies
     commsData.forEach((obj: IObj) => {
       const status = obj.responsestatus;
-      console.log(status);
-      // if(status === 200) responseCodes['200-299'] += 1;
       if (status >= 500) {
         responseCodes['500-599'] += 1;
       } else if (status >= 400) {

app/charts/RouteChart.jsx

@@ -5,7 +5,6 @@ import Graph from 'react-graph-vis';
 
 const RouteChart = React.memo(() => {
   const communicationsData = useContext(CommsContext).commsData;
-  console.log(communicationsData);
   // gather all communicationsData and sort them using matching correlatingid.
   // resObj { key = correlatingid : value = array of objects{ microservice , time} }
   // resObj { key = correlatingid : value = array of objects{ microservice , time, functionName} }

app/components/Occupied.tsx

@@ -54,12 +54,11 @@ const Occupied = React.memo(() => {
   const [searchTerm, setSearchTerm] = useState<string>('Search...');
   // Dynamic refs
   const delRef = useRef<any>([]);
-  //check context from dashboard
   useEffect(() => {
     setServicesData([]);
     getApplications(); 
   }, []);
-  console.log(mode);
+
 
   // Ask user for deletetion confirmation
   const confirmDelete = (event: ClickEvent, app: string, i: number) => {

app/components/Settings.tsx

@@ -7,7 +7,6 @@ import { DashboardContext } from '../context/DashboardContext';
 // Need to add flag to turn off the splash at start
 // Need to add flag to turn off getting started page
 // Need to add flag to turn on/off live data (ideally persist on restart)
-// Need to add dark mode
 
 //Typescript
 type ClickEvent = React.MouseEvent<HTMLElement>;

app/components/Styling.tsx

@@ -11,13 +11,10 @@ export const lightAndDark = {
     color: "#444d56",
   },
   darkModeHeader : {
-    // backgroundColor: "#transparent",
     backgroundColor: "#181818",
     color: "white",
   },
   darkModeMain : {
-    // backgroundImage: "url('../assets/mountain_longer.png')",
-    // backgroundColor: "#transparent",
     backgroundColor: "#181818",
     backgroundSize: 'contain',
     flex: "1",

chronos_npm_package/chronos.js

@@ -101,7 +101,16 @@ chronos.track = () => {
   }
   return null;
 };
-
+/**
+ * Wraps the gRPC server object to automatically write logs to user configed DB
+ *
+ * If the provided database is MongoDB, connection will be made to the Mongodb Atlas
+ *
+ * If the provided database is PostgreSQL, connection will be made to PostgreSQL client
+ * @param {*} server
+ * @param {*} proto
+ * @param {*} methods
+ */
 chronos.ServerWrapper = (server, proto, methods) => {
   const { database } = userConfig;
   if (database.type === 'MongoDB') {
@@ -112,7 +121,16 @@ chronos.ServerWrapper = (server, proto, methods) => {
   }
   return null;
 };
-
+/**
+ * Wraps the gRPC client to automatically write logs to user configed DB
+ *
+ * If the provided database is MongoDB, connection will be made to the Mongodb Atlas
+ *
+ * If the provided database is PostgreSQL, connection will be made to PostgreSQL client
+ *
+ * @param {*} client
+ * @param {*} service
+ */
 chronos.ClientWrapper = (client, service) => {
   const { database } = userConfig;
   if (database.type === 'MongoDB') {
@@ -124,6 +142,12 @@ chronos.ClientWrapper = (client, service) => {
   return null;
 };
 
+/**
+ * Allows the passthrough of metadata from gRPC server to gRPC client
+ *
+ * @param {*} client
+ * @param {*} servere
+ */
 chronos.link = (client, server) => {
   client.metadata = server.metadataHolder;
 };

chronos_npm_package/controllers/helpers.js

@@ -5,7 +5,7 @@ const helpers = {
    * Sets the default interval to 5 seconds and dockerized to false
    */
   validateInput(config) {
-    let { microservice, database, interval, dockerized, connection } = config;
+    const { microservice, database, interval, dockerized, connection } = config;
 
     // Validate all required fields exist and are valid input types
     if (!microservice || typeof microservice !== 'string') {
@@ -20,7 +20,9 @@ const helpers = {
       throw new Error('Invalid input "database URI": Please provide the URI to your database');
     }
     if (!database.connection || typeof database.connection !== 'string') {
-      throw new Error('Invalid input "database connection type: Please provide the type of connection');
+      throw new Error(
+        'Invalid input "database connection type: Please provide the type of connection'
+      );
     }
 
     // Validate database type
@@ -32,7 +34,7 @@ const helpers = {
     if (database.connection !== 'REST' && database.connection !== 'gRPC') {
       throw new Error(
         `Invalid database connection "${database.connection}". Chronos only supports REST and gRPC.`
-      )
+      );
     }
 
     // Default interval to one minute
@@ -55,7 +57,7 @@ const helpers = {
 
       // Setup notifications for user
       notifications.forEach(obj => {
-        const type = obj.type;
+        const { type } = obj;
 
         // Throw errors on unsupported notification methods
         if (!features.includes(type)) {

electron/Main.ts

@@ -1,18 +1,9 @@
 import { app, BrowserWindow } from 'electron';
-// import { REACT_DEVELOPER_TOOLS } from 'electron-devtools-installer';
-// const { default: installExtension } = require('electron-devtools-installer');
+
 import path from 'path';
 import './routes/dashboard';
 import './routes/data';
-// require('./routes/dashboard');
-// require('./routes/data');
-
-// Install React Dev Tools
-// app.whenReady().then(() => {
-//   installExtension(REACT_DEVELOPER_TOOLS)
-//     .then((name: string) => console.log(`Added Extension:  ${name}`))
-//     .catch((err: Error) => console.log('An error occurred: ', err));
-// });
+
 
 // Declare variable to be used as the application window
 let win: Electron.BrowserWindow;

electron/routes/data.ts

@@ -21,7 +21,6 @@ let currentDatabaseType: string;
  */
 ipcMain.on('connect', async (message: Electron.IpcMainEvent, index: number) => {
   try {
-    // console.log('electron/routes/data.ts, ipcMain.on(connect): 1 Attempting to connect to DB');
 
     // Extract databaseType and URI from settings.json at particular index
     // get index from application context
@@ -54,7 +53,7 @@ ipcMain.on('connect', async (message: Electron.IpcMainEvent, index: number) => {
  */
 ipcMain.on('servicesRequest', async (message: Electron.IpcMainEvent) => {
   try {
-    // console.log('electron/routes/data.ts, ipcMain.on(servicesRequest): 4 Requesting application microservices');
+   
 
     let result: any;
 
@@ -85,7 +84,7 @@ ipcMain.on('servicesRequest', async (message: Electron.IpcMainEvent) => {
  */
 ipcMain.on('commsRequest', async (message: Electron.IpcMainEvent) => {
   try {
-    // console.log(`Requesting communication data`)
+  
 
     let result: any;
 
@@ -118,7 +117,6 @@ ipcMain.on('commsRequest', async (message: Electron.IpcMainEvent) => {
  */
 ipcMain.on('healthRequest', async (message: Electron.IpcMainEvent, service: string) => {
   try {
-    // console.log(`Requesting microservice health for "${service}"`)
 
     let result: any;
 
@@ -161,7 +159,6 @@ ipcMain.on('healthRequest', async (message: Electron.IpcMainEvent, service: stri
  */
 ipcMain.on('dockerRequest', async (message, service) => {
   try {
-    // console.log(`Requesting container information for "${service}"`)
 
     let result: any;
     // Mongo Database