Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
101 changes: 47 additions & 54 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@
"mongoose": "^5.7.5"
},
"devDependencies": {
"nodemon": "^2.0.4"
"nodemon": "^2.0.7"
}
}
1 change: 1 addition & 0 deletions routes/todoRoute.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
const express = require('express');
const TodoController = require('../todosController/todos');
const Todo = require('./../models/Todo');

const router = express.Router();

Expand Down
28 changes: 15 additions & 13 deletions todosController/todos.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,19 +35,21 @@ class TodosController {

async createTodo(req, res) {
let newTodo = new Todo(req.body);
let title = req.body.title;
let body = req.body.body;
if (!req.body.title) {
return res.status(400).send({
success: 'false',
message: 'title is required',
success: "false",
message: "title is required",
});
}
try {
const todo = await newTodo.save();
const todo = await newTodo.save({title, body});
res.status(200).json(todo);
} catch (error) {
res.status(400).send({
success: 'false',
message: 'Unable to save to database',
success: "false",
message: "title is required",
});
}
}
Expand All @@ -58,14 +60,14 @@ class TodosController {

if(!ObjectId.isValid(id) && !id.match(/^[a-fA-F0-9]{24}$/)){
return res.status(404).send({
success: 'false',
message: 'todo does not exist',
success: "false",
message: "todo does not exist",
});
}
if (!req.body.title) {
return res.status(400).send({
success: 'false',
message: 'title is required',
success: "false",
message: "title is required",
});
}

Expand All @@ -82,17 +84,17 @@ class TodosController {
const {id} = req.params;
if(!ObjectId.isValid(id) && !id.match(/^[a-fA-F0-9]{24}$/)){
return res.status(404).send({
success: 'false',
message: 'todo does not exist',
success: "false",
message: "todo does not exist",
});
}
try {
const deleted = await Todo.findByIdAndRemove(id);
return res.status(200).json(deleted);
} catch (error) {
res.status(400).send({
success: 'false',
message: 'Unable to delete from database',
success: "false",
message: "Unable to delete from database",
});
}
}
Expand Down