From cb097bba12512769142228f8806b5b3c4c0e424f Mon Sep 17 00:00:00 2001 From: Mohamud E Hasen Date: Tue, 29 Jul 2025 12:55:21 -0500 Subject: [PATCH] added week18 assignment --- data/tasks.json | 30 ++++++--- routes/tasks.js | 172 +++++++++++++++++++++++++++++++++++++++--------- server.js | 1 - 3 files changed, 163 insertions(+), 40 deletions(-) diff --git a/data/tasks.json b/data/tasks.json index 3747a16..035f5fa 100644 --- a/data/tasks.json +++ b/data/tasks.json @@ -1,7 +1,7 @@ [ { "id": "1", - "title": "Learn Node.js Fundamentals", + "title": "Learn php ", "description": "Complete the Node.js basics course and build a simple server", "status": "in-progress", "priority": "high", @@ -38,8 +38,8 @@ }, { "id": "2", - "title": "Build REST API", - "description": "Create a complete REST API with CRUD operations", + "title": "Learn php ", + "description": "Complete the Node.js basics course and build a simple server", "status": "pending", "priority": "medium", "dueDate": "2024-01-20", @@ -142,15 +142,27 @@ "updatedAt": "2024-01-04T13:20:00.000Z" }, { - "id": "5", - "title": "Test", - "description": "Test", - "status": "pending", + "id": 6, + "title": "Learn Node.js Fundamentals", + "description": "Complete the Node.js basics course and build a simple server", + "status": "in-progress", + "priority": "high", + "dueDate": null, + "assignedTo": null, + "subtasks": [], + "createdAt": "2025-07-28T18:45:21.961Z", + "updatedAt": "2025-07-28T18:45:21.961Z" + }, + { + "id": 7, + "title": "Learn express ", + "description": "Complete the Node.js basics course and build a simple server", + "status": "in-progress", "priority": "medium", "dueDate": null, "assignedTo": null, "subtasks": [], - "createdAt": "2025-07-23T04:22:47.898Z", - "updatedAt": "2025-07-23T04:22:47.899Z" + "createdAt": "2025-07-28T18:46:54.199Z", + "updatedAt": "2025-07-28T18:46:54.200Z" } ] \ No newline at end of file diff --git a/routes/tasks.js b/routes/tasks.js index b2bcfcb..7d47714 100644 --- a/routes/tasks.js +++ b/routes/tasks.js @@ -153,80 +153,192 @@ router.post("/tasks", async (req, res) => { try { // TODO: Implement task creation // 1. Extract data from req.body (title, description, status, priority, etc.) + const { title, description, status, priority } = req.body; + // 2. Validate the data using validateTaskData function + + if (!title || !description || !status || !priority) { + return res.status(400).json({ error: "Missing reqiired fild" }); + } + // 3. Get all existing tasks using getAllTasks() + const tasks = await getAllTasks(); + // 4. Generate a new ID for the task + const newId = + tasks.length > 0 + ? Math.max(...tasks.map((task) => parseInt(task.id))) + 1 + : 1; + // 5. Create a new task object with all required fields + const newTask = { + id: newId, + title: title, + description: description, + status: status, + priority: priority, + dueDate: null, + assignedTo: null, + subtasks: [], + createdAt: new Date().toISOString(), + updatedAt: new Date().toISOString(), + }; + // 6. Add the task to the tasks array + tasks.push(newTask); + // 7. Save to file using writeTasks() + await writeTasks(tasks); + // 8. Send success response with status 201 + res.status(201).json({ + success: true, + message: "create task succesfully", + data: newTask, + }); // Temporary response - remove this when you implement the above - res.status(501).json({ - success: false, - error: - "POST endpoint not implemented yet - implement task creation above", - }); } catch (error) { res.status(500).json({ success: false, error: "Error creating task", - }); // 500 = Server Error + }); // 500 = Server Error } }); -// PUT /api/tasks/:id - Update task -// PUT requests are used to update existing resources -// The entire resource is replaced with the new data router.put("/tasks/:id", async (req, res) => { try { // TODO: Implement task update // 1. Extract the task ID from req.params + const { id } = req.params; + // console.log(id) // 2. Get the update data from req.body + const { title, description } = req.body; + // console.log( title,description) + // const updateData = req.body; + // 3. Validate the data if status or priority is being updated + if (!title || !description) { + return res.status(400).json({ error: "Missing reqiired fild" }); + } + const tasks = await getAllTasks(); // 4. Get all tasks and find the task by ID + const taskIndex = tasks.findIndex((task) => String(task.id) === id); + if (taskIndex === -1) { + return res + .status(404) + .json({ success: false, message: "Task not found" }); + } + + console.log(tasks); + // 5. Check if task exists, return 404 if not found + // 6. Update the task with new data + const newData = { title: "title", description: "description" }; + if (title) tasks[taskIndex].title = title; + if (description) tasks[taskIndex].description = description; + // 7. Save to file using writeTasks() + await writeTasks(tasks); + // 8. Send success response with the updated task - // Temporary response - remove this when you implement the above - res.status(501).json({ - success: false, - error: "PUT endpoint not implemented yet - implement task update above", + res.status(200).json({ + success: true, + message: "user updated successfully", + data: tasks[taskIndex], }); } catch (error) { res.status(500).json({ success: false, - error: "Error updating task", - }); // 500 = Server Error + error: error, + }); // 500 = Server Error } }); - // DELETE /api/tasks/:id - Delete task // DELETE requests are used to remove resources +// router.delete("/tasks/:id", async (req, res) => { +// try { +// // TODO: Implement task deletion +// // 1. Extract the task ID from req.params +// // 2. Get all tasks and find the task by ID +// // 3. Check if task exists, return 404 if not found +// // 4. Store the task before deletion (for response) +// // 5. Remove the task from the array +// // 6. Save to file using writeTasks() +// // 7. Send success response with the deleted task + +// // Temporary response - remove this when you implement the above +// // res.status(501).json({ +// // success: false, +// // error: +// // "DELETE endpoint not implemented yet - implement task deletion above", +// // }); +// // } catch (error) { +// // res.status(500).json({ +// // success: false, +// // error: "Error deleting task", +// // }); // 500 = Server Error + +// const taskId = req.params.id; +// const tasks = readTasks(); + +// const taskIndex = tasks.findIndex((task) => task.id === taskId); + +// if (taskIndex === -1) { +// return res.status(404).json({ +// success: false, +// message: "Task not found", +// }); +// } + +// const deletedTask = tasks[taskIndex]; +// tasks.splice(taskIndex, 1); + +// writeTasks(tasks); + +// res.status(200).json({ +// success: true, +// message: "Task deleted successfully", +// data: deletedTask, +// }); +// } catch (error) { +// res.status(500).json({ +// success: false, +// error: "Error deleting task", +// }); +// } +// }); +// DELETE /api/tasks/:id - Delete task by ID router.delete("/tasks/:id", async (req, res) => { try { - // TODO: Implement task deletion - // 1. Extract the task ID from req.params - // 2. Get all tasks and find the task by ID - // 3. Check if task exists, return 404 if not found - // 4. Store the task before deletion (for response) - // 5. Remove the task from the array - // 6. Save to file using writeTasks() - // 7. Send success response with the deleted task + const { id } = req.params; - // Temporary response - remove this when you implement the above - res.status(501).json({ - success: false, - error: - "DELETE endpoint not implemented yet - implement task deletion above", + const tasks = await getAllTasks(); // Read all tasks from file + const taskIndex = tasks.findIndex((task) => String(task.id) === id); // Match ID as string + + if (taskIndex === -1) { + return res.status(404).json({ + success: false, + message: "Task not found", + }); + } + + const deletedTask = tasks.splice(taskIndex, 1)[0]; // Remove and store the deleted task + + await writeTasks(tasks); // Write updated task list back to file + + res.status(200).json({ + success: true, + message: "Task deleted successfully", + data: deletedTask, }); } catch (error) { res.status(500).json({ success: false, error: "Error deleting task", - }); // 500 = Server Error + }); } }); diff --git a/server.js b/server.js index 47a9f9a..c249d52 100644 --- a/server.js +++ b/server.js @@ -14,7 +14,6 @@ app.use(cors()); // Enable CORS for all routes app.use(bodyParser.json()); // Parse JSON request bodies app.use(bodyParser.urlencoded({ extended: true })); // Parse URL-encoded bodies - // Mount API routes app.use("/api", taskRoutes);