Browse Source

Delete is working. Can handle /todos and /todos/ for list all.

Charlie Root 7 years ago
parent
commit
03cbc6c831
2 changed files with 5 additions and 3 deletions
  1. 5 3
      sql_todo.py
  2. BIN
      todo.db

+ 5 - 3
sql_todo.py

@@ -71,8 +71,9 @@ class TodoPath(Resource):
         return { 'id': t.id, 'name': t.name, 'detail': t.detail }
 
     def delete(self, todo_id):
-        abort_if_todo_doesnt_exist(todo_id)
-        del TODOS[todo_id]
+        t = abort_if_todo_doesnt_exist(todo_id)
+        db.session.delete(t)
+        db.session.commit()
         return '', 204
 
     # @auth.login_required
@@ -105,7 +106,8 @@ class TodoList(Resource):
 ##
 ## Actually setup the Api resource routing here
 ##
-api.add_resource(TodoList, '/todos')
+api.add_resource(TodoList, '/todos', '/todos/')
+# api.add_resource(TodoList, '/todos/')
 api.add_resource(TodoPath, '/todos/<todo_id>')
 
 

BIN
todo.db