Implemented ability to delete TODOs

Author: smashdevcode

app/scripts/controllers/todo.js

@@ -3,8 +3,9 @@
 function TodoCtrl ($scope, dataService) {
 
   $scope.deleteTodo = function(todo, index) {
-    $scope.todos.splice(index, 1);
-    dataService.deleteTodo(todo);
+    dataService.deleteTodo(todo).then(function() {
+      $scope.todos.splice(index, 1);
+    });
   };
 
   $scope.saveTodos = function() {

app/scripts/services/data.js

@@ -7,22 +7,27 @@ function DataService ($http, $q) {
   };
 
   this.deleteTodo = function(todo) {
-    console.log("I deleted the " + todo.name + " todo!");
+    if (!todo._id) {
+      return $q.resolve();
+    }
+    return $http.delete('/api/todos/' + todo._id).then(function() {
+      console.log("I deleted the " + todo.name + " todo!");
+    });
   };
 
   this.saveTodos = function(todos) {
     var queue = [];
     todos.forEach(function(todo) {
-        var request;
-        if(!todo._id) {
-          request = $http.post('/api/todos', todo);
-        } else {
-          request = $http.put('/api/todos/' + todo._id, todo).then(function(result) {
-            todo = result.data.todo;
-            return todo;
-          });
-        }
-        queue.push(request);
+      var request;
+      if(!todo._id) {
+        request = $http.post('/api/todos', todo);
+      } else {
+        request = $http.put('/api/todos/' + todo._id, todo).then(function(result) {
+          todo = result.data.todo;
+          return todo;
+        });
+      }
+      queue.push(request);
     });
     return $q.all(queue).then(function(results) {
       console.log("I saved " + todos.length + " todos!");

src/api/index.js

@@ -38,6 +38,14 @@ router.put('/todos/:id', function(req, res) {
   });
 });
 
-// TODO: Add DELETE route to remove existing entries
+router.delete('/todos/:id', function(req, res) {
+  var id = req.params.id;
+  Todo.findByIdAndRemove(id, function(err, result) {
+    if (err) {
+      return res.status(500).json({ err: err.message });
+    }
+    res.json({ message: 'Todo Deleted' });
+  });
+});
 
 module.exports = router;