Open cpiazza opened 3 years ago
The success method is being shown regardless of if the above updates succeed or not. Similar pattern exists in other methods in this controller where .update is called - which will fail silently. Please fix.
.update
https://github.com/vishal-gh/TodoApp/blob/da395eb5f57e2bf966c7fe618bc7f8f2a99a62ec/app/controllers/todo_items_controller.rb#L32
Fixed.
vishal-gh
commented
3 years ago
The success method is being shown regardless of if the above updates succeed or not. Similar pattern exists in other methods in this controller where
.updateis called - which will fail silently. Please fix.https://github.com/vishal-gh/TodoApp/blob/da395eb5f57e2bf966c7fe618bc7f8f2a99a62ec/app/controllers/todo_items_controller.rb#L32