From b7f3f31fb9f00b5445c3f5b98c454934927c1dc7 Mon Sep 17 00:00:00 2001
From: Mallepally Lokeshwar Reddy <lokeshwar777@gmail.com>
Date: Sat, 15 Jun 2024 13:56:44 +0530
Subject: [PATCH 1/3] chore: Replace useState with useEffect in Comment
 component

---
 src/components/TutorialPage/components/Commnets/Comment.jsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/TutorialPage/components/Commnets/Comment.jsx b/src/components/TutorialPage/components/Commnets/Comment.jsx
index 09e486fc..c742ae16 100644
--- a/src/components/TutorialPage/components/Commnets/Comment.jsx
+++ b/src/components/TutorialPage/components/Commnets/Comment.jsx
@@ -60,7 +60,7 @@ const Comment = ({ id }) => {
   const firestore = useFirestore();
   const firebase = useFirebase();
   const dispatch = useDispatch();
-  useState(() => {
+  useEffect(() => {
     getCommentData(id)(firebase, firestore, dispatch);
   }, [id]);
 
@@ -165,7 +165,7 @@ const Comment = ({ id }) => {
           <div style={{ margin: "10px 0 0 10px" }}>
             <Textbox type="reply" handleSubmit={handleSubmit} />
             {replies?.replies.map((id, index) => {
-              return <Comment id={id} />;
+              return <Comment key={index} id={id} />;
             })}
           </div>
         )}

From 50ecf92ab8a8e0fe8a904929aa9bf5676d876cbd Mon Sep 17 00:00:00 2001
From: Mallepally Lokeshwar Reddy <lokeshwar777@gmail.com>
Date: Sat, 15 Jun 2024 14:00:18 +0530
Subject: [PATCH 2/3] chore: Refactor Textbox component to handle comment
 submission on Enter key press

---
 .../TutorialPage/components/Commnets/Textbox.jsx   | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/components/TutorialPage/components/Commnets/Textbox.jsx b/src/components/TutorialPage/components/Commnets/Textbox.jsx
index 891d9c00..998011da 100644
--- a/src/components/TutorialPage/components/Commnets/Textbox.jsx
+++ b/src/components/TutorialPage/components/Commnets/Textbox.jsx
@@ -1,4 +1,4 @@
-import { useState } from "react";
+import React, { useState } from "react";
 import {
   Box,
   TextField,
@@ -40,6 +40,13 @@ const Textbox = ({ type, handleSubmit }) => {
         fullWidth
         value={commentText}
         onChange={e => setCommentText(e.target.value)}
+        onKeyDown={e => {
+          if (e.key === "Enter") {
+            e.preventDefault();
+            handleSubmit(commentText);
+            setCommentText("");
+          }
+        }}
         InputProps={{
           endAdornment: (
             <InputAdornment position="end" sx={{ position: "relative" }}>
@@ -70,7 +77,10 @@ const Textbox = ({ type, handleSubmit }) => {
       <Button
         variant="contained"
         disableElevation
-        onClick={() => handleSubmit(commentText)}
+        onClick={() => {
+          handleSubmit(commentText);
+          setCommentText("");
+        }}
       >
         <Send />
       </Button>

From 0336425a289d8ee27251ef9c2a93d2a8772e2681 Mon Sep 17 00:00:00 2001
From: Mallepally Lokeshwar Reddy <lokeshwar777@gmail.com>
Date: Sat, 15 Jun 2024 14:17:29 +0530
Subject: [PATCH 3/3] refactor(CommentBox): Improvements in handling comment
 submission

---
 .../components/Commnets/CommentBox.jsx        | 29 ++++----------
 src/components/TutorialPage/index.jsx         | 35 ++++++++++++++++-
 src/store/actions/tutorialPageActions.js      | 38 +++++++++++--------
 3 files changed, 62 insertions(+), 40 deletions(-)

diff --git a/src/components/TutorialPage/components/Commnets/CommentBox.jsx b/src/components/TutorialPage/components/Commnets/CommentBox.jsx
index 02f462f9..87b35387 100644
--- a/src/components/TutorialPage/components/Commnets/CommentBox.jsx
+++ b/src/components/TutorialPage/components/Commnets/CommentBox.jsx
@@ -3,9 +3,7 @@ import { makeStyles } from "@mui/styles";
 import React, { useEffect, useState } from "react";
 import Textbox from "./Textbox";
 import Comment from "./Comment";
-import { addComment } from "../../../../store/actions/tutorialPageActions";
-import { useDispatch, useSelector } from "react-redux";
-import { useFirebase, useFirestore } from "react-redux-firebase";
+
 const useStyles = makeStyles(() => ({
   container: {
     margin: "10px 0",
@@ -28,23 +26,10 @@ const useStyles = makeStyles(() => ({
   }
 }));
 
-const CommentBox = ({ commentsArray, tutorialId }) => {
+const CommentBox = ({ commentsArray, onAddComment }) => {
   const classes = useStyles();
-  const firestore = useFirestore();
-  const firebase = useFirebase();
-  const dispatch = useDispatch();
-  const [comments, setComments] = useState([]);
+  const [comments, setComments] = useState(commentsArray);
   const [currCommentCount, setCurrCommentCount] = useState(3);
-  const handleSubmit = comment => {
-    const commentData = {
-      content: comment,
-      replyTo: tutorialId,
-      tutorial_id: tutorialId,
-      createdAt: firestore.FieldValue.serverTimestamp(),
-      userId: "codelabzuser"
-    };
-    addComment(commentData)(firebase, firestore, dispatch);
-  };
 
   useEffect(() => {
     setComments(commentsArray?.slice(0, currCommentCount));
@@ -65,17 +50,17 @@ const CommentBox = ({ commentsArray, tutorialId }) => {
       <Typography variant="h5" sx={{ fontWeight: "600" }}>
         Comments({commentsArray?.length || 0})
       </Typography>
-      <Textbox handleSubmit={handleSubmit} />
+      <Textbox handleSubmit={onAddComment} />
       <Grid container rowSpacing={2}>
         {comments?.map((id, index) => {
           return (
-            <Grid item xs={12}>
-              <Comment id={id} key={index} />
+            <Grid item xs={12} key={index}>
+              <Comment id={id} />
             </Grid>
           );
         })}
         <Grid item container justifyContent="center">
-          {comments?.length != commentsArray?.length && (
+          {comments?.length < commentsArray?.length && (
             <Button
               sx={{ textTransform: "none", fontSize: "14px" }}
               onClick={increaseCommentCount}
diff --git a/src/components/TutorialPage/index.jsx b/src/components/TutorialPage/index.jsx
index dd7895c2..615e9f91 100644
--- a/src/components/TutorialPage/index.jsx
+++ b/src/components/TutorialPage/index.jsx
@@ -11,7 +11,8 @@ import StepsBar from "./StepBar";
 import useWindowSize from "../../helpers/customHooks/useWindowSize";
 import {
   getTutorialData,
-  getTutorialSteps
+  getTutorialSteps,
+  addComment
 } from "../../store/actions/tutorialPageActions";
 import { getUserProfileData } from "../../store/actions";
 import { useDispatch, useSelector } from "react-redux";
@@ -24,6 +25,7 @@ function TutorialPage({ background = "white", textColor = "black" }) {
   const history = useHistory();
   const windowSize = useWindowSize();
   const [openMenu, setOpen] = useState(false);
+  const [commentsArray, setCommentsArray] = useState([]);
   const toggleSlider = () => {
     setOpen(!openMenu);
   };
@@ -42,6 +44,11 @@ function TutorialPage({ background = "white", textColor = "black" }) {
       }
     }) => data
   );
+
+  useEffect(() => {
+    setCommentsArray(tutorial?.comments);
+  }, [tutorial?.comments]);
+
   const loading = useSelector(
     ({
       tutorialPage: {
@@ -72,6 +79,27 @@ function TutorialPage({ background = "white", textColor = "black" }) {
     history.push("/not-found");
   }
 
+  const handleAddComment = async comment => {
+    const commentData = {
+      content: comment,
+      replyTo: id,
+      tutorial_id: id,
+      createdAt: firestore.FieldValue.serverTimestamp(),
+      userId: "codelabzuser"
+    };
+    const commentId = await addComment(commentData)(
+      firebase,
+      firestore,
+      dispatch
+    );
+
+    if (!commentsArray || commentsArray?.length === 0) {
+      setCommentsArray([commentId]);
+    } else {
+      setCommentsArray(prevComments => [commentId, ...prevComments]);
+    }
+  };
+
   return (
     <Box
       className={classes.wrapper}
@@ -112,7 +140,10 @@ function TutorialPage({ background = "white", textColor = "black" }) {
         >
           <PostDetails details={postDetails} />
           <Tutorial steps={steps} />
-          <CommentBox commentsArray={tutorial?.comments} tutorialId={id} />
+          <CommentBox
+            commentsArray={commentsArray}
+            onAddComment={handleAddComment}
+          />
         </Grid>
 
         <Grid
diff --git a/src/store/actions/tutorialPageActions.js b/src/store/actions/tutorialPageActions.js
index c716765a..94ec69b5 100644
--- a/src/store/actions/tutorialPageActions.js
+++ b/src/store/actions/tutorialPageActions.js
@@ -120,6 +120,9 @@ export const getTutorialData =
         .doc(tutorialID)
         .get();
       const tutorial = data.data();
+      if (tutorial.comments && Array.isArray(tutorial.comments)) {
+        tutorial.comments.reverse();
+      }
       dispatch({ type: actions.GET_POST_DATA_SUCCESS, payload: tutorial });
     } catch (e) {
       dispatch({ type: actions.GET_POST_DATA_FAIL });
@@ -195,25 +198,28 @@ export const getCommentReply =
 export const addComment = comment => async (firebase, firestore, dispatch) => {
   try {
     dispatch({ type: actions.ADD_COMMENT_START });
+
+    const docref = await firestore
+      .collection("cl_comments")
+      .add(comment);
+
     await firestore
       .collection("cl_comments")
-      .add(comment)
-      .then(docref => {
-        firestore.collection("cl_comments").doc(docref.id).update({
-          comment_id: docref.id
-        });
-        if (comment.replyTo == comment.tutorial_id) {
-          firestore
-            .collection("tutorials")
-            .doc(comment.tutorial_id)
-            .update({
-              comments: firebase.firestore.FieldValue.arrayUnion(docref.id)
-            });
-        }
-      })
-      .then(() => {
-        dispatch({ type: actions.ADD_COMMENT_SUCCESS });
+      .doc(docref.id)
+      .update({
+        comment_id: docref.id
       });
+
+    if (comment.replyTo === comment.tutorial_id) {
+      await firestore
+        .collection("tutorials")
+        .doc(comment.tutorial_id)
+        .update({
+          comments: firebase.firestore.FieldValue.arrayUnion(docref.id)
+        });
+    }
+
+    dispatch({ type: actions.ADD_COMMENT_SUCCESS });
   } catch (e) {
     dispatch({ type: actions.ADD_COMMENT_FAILED, payload: e.message });
   }