code cleaning

Author: johnlcos

server/controllers/boardsController.ts

@@ -6,7 +6,7 @@ import { BoardListItemState, BoardType } from "../../src/types.ts";
 const boardsController = {
   getMyBoards: async (req: Request, res: Response, next: NextFunction) => {
     try {
-      // find all boards beloning to the user, push board names into an array and save on locals
+      // find all boards from the user, push board names into an array and save on locals
       const user = await User.findOne({ _id: req.params.userId }).populate(
         "boards"
       );
@@ -144,7 +144,6 @@ const boardsController = {
       await Board.findOneAndDelete({
         _id: req.params.boardId,
       });
-      console.log("board deleted");
       return next();
     } catch (err) {
       // pass error through to global error handler
@@ -164,7 +163,6 @@ const boardsController = {
         },
         { new: true }
       );
-      console.log("editedBoard: ", editedBoard);
       res.locals.board = editedBoard;
       return next();
     } catch (err) {

server/controllers/userController.ts

@@ -41,11 +41,9 @@ const userController = {
 
   // middleware for verifying a user on login
   verifyUser: async (req: Request, res: Response, next: NextFunction) => {
-    // obtain user name password from request body
     // obtain user name password from request body
     const { username, password } = req.body;
     // check for a missing input
-    // check for a missing input
     if (!username || !password) {
       return next({
         log: "Missing username or password in verifyUser",
@@ -54,23 +52,19 @@ const userController = {
       });
     }
     try {
-      // search DB for the user based on the username
       // search DB for the user based on the username
       const user = await User.findOne({ username });
       // if now user is found error out
-      // if now user is found error out
       if (!user) {
         return next({
           log: `userController.verifyUser ERROR: no user with input username found in DB`,
           status: 400,
           message: { err: "Invalid username or password" },
         });
       } else {
-        // else a user is found, check passwords
         // else a user is found, check passwords
         const resultPassword = await bcrypt.compare(password, user.password);
         // if passwords do not match error out
-        // if passwords do not match error out
         if (!resultPassword) {
           return next({
             log: `userController.verifyUser ERROR: input password does not match stored password`,
@@ -89,12 +83,6 @@ const userController = {
         status: 500,
         message: { err: "Error occured creating user" },
       });
-      // send any errors to global error handler
-      return next({
-        log: `usersController.createUser ERROR: ${err}`,
-        status: 500,
-        message: { err: "Error occured creating user" },
-      });
     }
   },
 };

server/routes/boardsRouter.ts

@@ -18,6 +18,7 @@ router.get(
   }
 );
 
+// route for getting all tasks associated with a board
 router.get(
   "/board",
   boardsController.getCurrentBoard,
@@ -27,6 +28,7 @@ router.get(
   }
 );
 
+// route for creating a new baord
 router.post(
   "/create",
   boardsController.createBoard,

src/components/Column.tsx

@@ -11,29 +11,19 @@ const Column = ({
   boardState,
   setBoardState,
 }: ColumnProps) => {
-  // state for rendering new task modal
   const [addingTask, setAddingTask] = useState<boolean>(false);
-
-  // state for number of tasks in a column
   const [numTasks, setNumTasks] = useState<number>(0);
-
-  // state for storing task card components
   const [taskCards, setTaskCards] = useState<ReactNode[]>([]);
-
-  // state for editing a task card
   const [editingTask, setEditingTask] = useState<TaskState | null>(null);
 
-  // render new task modal on button click
   const handleNewTask = () => {
     setAddingTask(true);
   };
 
   //effect for rendering cards
   useEffect(() => {
     const column = boardState[name];
-    console.log("Column column: ", column);
     setNumTasks(column.length);
-    // map column to an array of task card components and then set as the state
     const cardsArray = column.map((task: TaskState) => {
       return (
         <Card info={task} setEditingTask={setEditingTask} key={task._id} />

src/components/CreateBoardModal.tsx

@@ -34,11 +34,9 @@ const CreateBoardModal = ({
       },
       body: JSON.stringify(body),
     });
-    // receive username and id from backend
-    // if request success, save username to state and route to dashboard
+    // receive board name and id from backend
     if (response.status === 200) {
       const responseData = await response.json();
-      console.log("Board created: ", responseData);
       const newBoardListItem = (
         <button
           className={`board-selector ${
@@ -61,7 +59,7 @@ const CreateBoardModal = ({
     }
   };
 
-  const isButtonDisabled: boolean = boardName === ""; //checking if boardName is empty? using trim in handle input
+  const isButtonDisabled: boolean = boardName === "";
 
   return createPortal(
     <div className="modal-overlay">

src/components/EditBoardModal.tsx

@@ -29,7 +29,7 @@ const EditBoardModal = ({
       },
       body: JSON.stringify(body),
     });
-    // if request success, update currentBoard
+    // receive board name and id from backend
     if (response.status === 200) {
       const responseData = await response.json();
       setCurrentBoard({ name: responseData.name, id: responseData._id });
@@ -58,7 +58,7 @@ const EditBoardModal = ({
     fetchDeleteBoard().catch(console.error);
   };
 
-  const isButtonDisabled: boolean = boardName === ""; //checking if boardName is empty? using trim in handle input
+  const isButtonDisabled: boolean = boardName === "";
 
   return createPortal(
     <div className="modal-overlay">

src/components/EditTaskModal.tsx

@@ -24,7 +24,7 @@ const EditTaskModal = ({
   const handleFormSubmit = (e: React.FormEvent<HTMLFormElement>) => {
     e.preventDefault;
     console.log("Edit Task Form Submitted: ", formData);
-    // send POST request with the new task card, edited task and current board
+    // send POST request with the  edited task, originla column and current board
     const fetchEditTask = async () => {
       const body = {
         ...formData,

src/containers/LeftContainer.tsx

@@ -8,13 +8,11 @@ const LeftContainer = ({
   setCurrentBoard,
   currentBoard,
 }: LeftContainerProps) => {
-  // creating state to open board creating modal
   const [creatingBoard, setCreatingBoard] = useState<boolean>(false);
-  // creating state to store the list of board names
   const [boardList, setBoardList] = useState<ReactNode[]>([]);
-  //state for selected board
   const [selectedBoard, setSelectedBoard] = useState<string | null>(null);
-  // make a request for all board naames, return an array containing strings of the board names
+
+  // make a request for all board names, return an array containing strings of the board names
   useEffect(() => {
     const fetchBoardList = async () => {
       const reponse: Response = await fetch(`/boards/myboards/${user.id}`);
@@ -35,10 +33,8 @@ const LeftContainer = ({
       setBoardList(boardSelectors);
     };
     fetchBoardList().catch(console.error);
-    // iterate through board names push buttons or components into array boardlist in state
   }, [currentBoard]);
 
-  // function for changing board when click selection button
   const handleBoardSelect = (e: React.MouseEvent<HTMLButtonElement>) => {
     e.preventDefault();
     setCurrentBoard({
@@ -48,9 +44,7 @@ const LeftContainer = ({
     setSelectedBoard(e.currentTarget.id);
   };
 
-  // function for creating a new board
   const handleCreateBoard = () => {
-    // create a pop up to create the new board
     setCreatingBoard(true);
   };
 

src/containers/MainContainer.tsx

@@ -17,7 +17,6 @@ const MainContainer = ({
     inReview: [],
     completed: [],
   });
-
   const [editingBoard, setEditingBoard] = useState<boolean>(false);
 
   // effect for fetching the current board info whenever currentBoard changes
@@ -29,8 +28,6 @@ const MainContainer = ({
           `/boards/board?board=${currentBoard.id}&user=${user.id}`
         );
         const board = await reponse.json();
-        // update state with the fetched data
-        console.log("MainContainer board: ", board);
         // eslint-disable-next-line @typescript-eslint/no-unused-vars
         const { boardOwner, name, _id, __v, ...columns } = board;
         await setBoardState({ ...columns });
@@ -39,11 +36,6 @@ const MainContainer = ({
     fetchBoard().catch(console.error);
   }, [currentBoard]);
 
-  //
-  useEffect(() => {
-    console.log("MainContainer boardState:", boardState); // Log the updated state after it's set
-  }, [boardState]); // Log when the boardState changes
-
   if (!currentBoard?.name) {
     return (
       <div className="main-container">

src/routes/Dashboard.tsx

@@ -5,12 +5,10 @@ import { useState } from "react";
 import "../scss/dashBoard.scss";
 
 const Dashboard = ({ user }: DashboardProps) => {
-  // state of current board selected
   const [currentBoard, setCurrentBoard] = useState<CurrentBoardState>({
     name: "",
     id: "",
   });
-  //state confirmDelete false
   return (
     <div className="main-page">
       <LeftContainer