[TCES-68] Create Job Posts Page: Replace mock data with backend endpo…

…ints (#137) * Task completed - Save as Draft and Publish buttons now call backend and not mockresponse - Buttons now redirect to job posts page and not dashboard - Made route names more consistent * Updated Job Posting - description field is now properly updated when publish button is pushed * Published job posts now marked as active, and not drafts * Job post creations accurately reflect draft or active * Removed unnecessary log * fix job_type and close_date field updates Signed-off-by: Kashish Mittal <[email protected]> --------- Signed-off-by: Kashish Mittal <[email protected]> Co-authored-by: Kashish Mittal <[email protected]>
uoftblueprint · Jan 20, 2025 · 11b20a8 · 11b20a8
1 parent 308229e
commit 11b20a8
Show file tree

Hide file tree

Showing 8 changed files with 25 additions and 26 deletions.
diff --git a/client/src/App.jsx b/client/src/App.jsx
@@ -371,7 +371,7 @@ function App() {
               }
             />
             <Route
-              path="/job-posts/add"
+              path="/job-postings/add"
               element={
                 <AuthGuard
                   isAuthenticated={isAuthenticated}

diff --git a/client/src/components/add-job-post-component/addJobPosting.jsx b/client/src/components/add-job-post-component/addJobPosting.jsx
@@ -17,7 +17,8 @@ function AddJobPostParent({ currUser }) {
       creation_date: dayjs(),
       close_date: dayjs().add(1, "month"),
       job_type: [],
-      description: "",
+      job_description: "",
+      state: ""
     },
     applicationFields: {
       custom_questions: [],

diff --git a/client/src/components/add-job-post-component/index.jsx b/client/src/components/add-job-post-component/index.jsx
@@ -17,7 +17,8 @@ import AddJobDetails from "./job-info-form";
 import AddApplicationFields from "./application-form-fields";
 import { Container, ButtonContainer } from "./index.styles";
 import UserType from "../../prop-types/UserType";
-import createJobPost from "./mockResponse";
+import { createJobPost } from "../../utils/job_posts_api";
+// import createJobPost from "./mockResponse";
 import PostingResultDialog from "./posting-result-dialog";
 
 function AddJobPost({ jobPostData, updateJobPostData, currUser }) {
@@ -33,7 +34,7 @@ function AddJobPost({ jobPostData, updateJobPostData, currUser }) {
   const SUCCESS_DRAFT = {
     isSuccess: true,
     message: "Your job posting was saved",
-    handleClose: () => navigate("/dashboard"),
+    handleClose: () => navigate("/job-postings"),
     buttonMessage: "CLOSE",
   };
   const ERROR_DRAFT = {
@@ -45,7 +46,7 @@ function AddJobPost({ jobPostData, updateJobPostData, currUser }) {
   const SUCCESS_PUBLISH = {
     isSuccess: true,
     message: "Your job posting was published",
-    handleClose: () => navigate("/dashboard"),
+    handleClose: () => navigate("/job-postings"),
     buttonMessage: "CLOSE",
   };
   const ERROR_PUBLISH = {
@@ -76,18 +77,17 @@ function AddJobPost({ jobPostData, updateJobPostData, currUser }) {
 
   const handleSubmit = async (e, postState) => {
     const DRAFT = "Draft";
+    jobPostData.jobInfo.state = postState // eslint-disable-line no-param-reassign
     const updatedJobPost = { ...jobPostData, state: postState };
-    updateJobPostData("state", postState);
 
     setIsLoading(true);
     try {
       const response = await createJobPost(
-        updatedJobPost,
+        updatedJobPost.jobInfo,
         currUser.userID,
         currUser.userID,
       );
-
-      if (response.ok) {
+      if (response.status === "success") {
         setResultModalValues(
           postState === DRAFT ? SUCCESS_DRAFT : SUCCESS_PUBLISH,
         );
@@ -231,7 +231,7 @@ function AddJobPost({ jobPostData, updateJobPostData, currUser }) {
                       variant="contained"
                       disabled={isLoading}
                       disableElevation
-                      onClick={() => handleSubmit(null, "Publish")}
+                      onClick={() => handleSubmit(null, "Active")}
                     >
                       PUBLISH
                     </Button>

diff --git a/client/src/components/add-job-post-component/job-info-form/index.jsx b/client/src/components/add-job-post-component/job-info-form/index.jsx
@@ -227,9 +227,7 @@ function AddJobDetails({ jobPostData, setJobPostData }) {
             sx={{ m: 1, width: "47%" }}
             value={jobPostData.close_date}
             minDate={dayjs()}
-            onChange={(newValue) =>
-              handleInputChange(newValue, jobPostData, "close_date")
-            }
+            onChange={(newValue) => handleInputChange(newValue, "close_date")}
             renderInput={(params) => (
               // eslint-disable-next-line
               <TextField {...params} error={false} helperText="" required />
@@ -243,14 +241,14 @@ function AddJobDetails({ jobPostData, setJobPostData }) {
         <TextField
           fullWidth
           sx={{ m: 1, width: "96%" }}
-          id="description"
+          id="job_description"
           label="Description"
           multiline
           rows={4}
-          value={jobPostData.description}
+          value={jobPostData.job_description}
           InputLabelProps={{ shrink: true, required: false }}
           helperText="*Required"
-          onChange={(e) => handleInputChange(e.target.value, "description")}
+          onChange={(e) => handleInputChange(e.target.value, "job_description")}
           required
         />
       </Container>

diff --git a/client/src/components/add-job-post-component/mockResponse.js b/client/src/components/add-job-post-component/mockResponse.js
@@ -6,14 +6,14 @@ function mockFetch(jobPostData, currUserId) {
   return new Promise((resolve, reject) => {
     console.log(reject);
     // Simulate error
-    setTimeout(() => {
-      reject(new Error("Failure"));
-    }, timeout);
-
-    // Simulate success
     // setTimeout(() => {
-    //   resolve({ ok: 200, jobPostData, currUserId });
+    //   reject(new Error("Failure"));
     // }, timeout);
+
+    // Simulate success
+    setTimeout(() => {
+      resolve({ ok: 200, jobPostData, currUserId });
+    }, timeout);
   });
 }
 

diff --git a/client/src/components/dashboard-component/header-dashboard-component/index.jsx b/client/src/components/dashboard-component/header-dashboard-component/index.jsx
@@ -89,7 +89,7 @@ function DashboardHeaderComponent({ currUser }) {
                 Add New Client
               </MenuItem>
               <MenuItem
-                onClick={() => handleClose("/job-posts/add")}
+                onClick={() => handleClose("/job-postings/add")}
                 sx={{ justifyContent: "center" }}
               >
                 Add New Job Posting

diff --git a/client/src/utils/constants.js b/client/src/utils/constants.js
@@ -14,7 +14,7 @@ export const COMPENSATION_RATES_FOR_JOB_POSTS = [
   "Hourly",
   "Weekly",
   "Annually",
-  "Commision",
+  "Commission",
   "Base and Commission",
 ];
 

diff --git a/server/src/controllers/job_posts/addJobPost.js b/server/src/controllers/job_posts/addJobPost.js
@@ -16,7 +16,7 @@ const addJobPostRequestHandler = async (req, res) => {
       close_date: closeDate,
       job_description: jobDescription,
       custom_questions: customQuestions,
-      state = "Draft",
+      state,
       // The creator field is excluded because the backend can retrieve the current user's ID from req.user.id.
     } = req.body;
 
@@ -85,7 +85,7 @@ const addJobPostRequestHandler = async (req, res) => {
       job_description: jobDescription,
       custom_questions: customQuestions,
       creator: req.user.id,
-      state: state === "Draft" ? "Draft" : "Active",
+      state,
     });
 
     // ! Return a response stating that the object is successfully created.