Merge pull request #103 from yennanliu/assignment-dev-07-more-biz-logic-on-codeReview-page

assignment-dev-07 : Add more biz logic on codeReview page

Author: yennanliu

springAssignmentSystem/README.md

@@ -53,6 +53,7 @@
 	- Part 27 : https://youtu.be/1s0oommR7qY?si=1g3zfpbJLFCMwM_V
 	- Part 28 : https://youtu.be/mEdWzE4-sLg?si=hzkrmArO8zjn7PXc
 	- Part 29 : https://youtu.be/utXVb3R1yuE?si=zbKXVhG7VMa214L4
+	- Part 30 : https://youtu.be/iYchh7v7P1Q?si=mDFJUFlhijuH83CN
 
 
 - Spring JWT

springAssignmentSystem/backend/SpringAssignmentSystem/src/main/java/com/yen/SpringAssignmentSystem/domain/Assignment.java

@@ -39,7 +39,7 @@ public class Assignment {
 
     //@ManyToOne(cascade = CascadeType.MERGE, fetch = FetchType.EAGER)
     //@Column(name="code_reviewer")
-    @ManyToOne
+    @ManyToOne(cascade = CascadeType.PERSIST) // https://blog.csdn.net/wsaicyj/article/details/123966389
     private User codeReviewer;
 
     @Column(name="submitted_date")

springAssignmentSystem/doc/progress.txt

@@ -85,4 +85,9 @@
 
 - 20231008
 		- https://youtu.be/mEdWzE4-sLg?si=hzkrmArO8zjn7PXc
-		- https://youtu.be/utXVb3R1yuE?si=zbKXVhG7VMa214L4
+		- https://youtu.be/utXVb3R1yuE?si=zbKXVhG7VMa214L4
+			- progress : 17:44
+
+- 20231009
+		- https://youtu.be/utXVb3R1yuE?si=zbKXVhG7VMa214L4
+		- https://youtu.be/iYchh7v7P1Q?si=mDFJUFlhijuH83CN

springAssignmentSystem/frontend/frontend_app/src/CodeReviewerDashBoard/index.js

@@ -15,10 +15,37 @@ const CodeReviewerDashboard = () => {
   const [getJwt, setJwt] = useLocalState("", "jwt"); // getter, setter
   const [assignments, setAssignment] = useState(null);
 
+  // https://youtu.be/utXVb3R1yuE?si=4Hv8qBS_gQ8mJVfF&t=408
+  function claimAssignment(assignment) {
+    //const decodedJwt = jwt_decode(getJwt);
+    const user = {
+      id: null,
+      username: "my_code_reivewer",
+      authorities: ["ROLE_CODE_REVIEWER"],
+    };
+    assignment.codeReviewer = user;
+    // TODO : fix this hardcode status
+    assignment.status = "In Review";
+    console.log("claim assignment : " + JSON.stringify(assignment));
+    ajax(`/api/assignments/${assignment.id}`, "PUT", getJwt, assignment).then(
+      (updatedAssignment) => {
+        // TODO : update view for assignment that changed
+        // copy an array syntax [...oldArray]
+        // https://youtu.be/utXVb3R1yuE?si=MkVbNb5R8imKjtkw&t=1947
+        const assignmentsCopied = [...assignments];
+        const i = assignmentsCopied.findIndex((x) => x.id === assignment.id);
+        assignmentsCopied[i] = updatedAssignment;
+        setAssignment(assignmentsCopied);
+      }
+    );
+  }
+
   useEffect(() => {
-    ajax("/api/assignments/", "GET", getJwt).then((assignmentsData) => {
-      setAssignment(assignmentsData);
-    });
+    ajax("/api/assignments/to_review", "GET", getJwt).then(
+      (assignmentsData) => {
+        setAssignment(assignmentsData);
+      }
+    );
   }, []);
 
   function createAssignment() {
@@ -108,10 +135,11 @@ const CodeReviewerDashboard = () => {
                   <Button
                     variant="secondary"
                     onClick={() => {
-                      navigate(`/assignments/${assignment.id}`);
+                      //navigate(`/assignments/${assignment.id}`);
+                      claimAssignment(assignment);
                     }}
                   >
-                    Edit
+                    Claim
                   </Button>
                 )}
               </Card.Body>

springAssignmentSystem/frontend/frontend_app/src/StatusBadge/index.js

@@ -11,6 +11,7 @@ const StatusBadge = (props) => {
     else if (text === "Needs Update") return "danger";
     else if (text === "Pending Submission") return "warning";
     else if (text === "Resubmitted") return "primary";
+    else if (text === "In Review") return "primary";
     else return "info";
   }
   return (