Skip to content

Commit 5f37d5b

Browse files
delete tests added and passing
1 parent cfaaa84 commit 5f37d5b

6 files changed

Lines changed: 1769 additions & 8 deletions

File tree

app/controllers/tasks_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ def update
7878
end
7979
end
8080

81-
def complete
81+
def toggle_complete
8282
begin
8383
@task = Task.find(params[:id])
8484
rescue

app/views/tasks/index.html.erb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,13 @@
44
<% @tasks.each do |task| %>
55
<li>
66
<% if task.completion_date %>
7-
<%= link_to "☑", complete_task_path(task.id), method: :patch %>
7+
<%= link_to "☑", toggle_complete_task_path(task.id), method: :patch %>
88
<s><%= link_to task.name, task_path(task.id) %></s>
99
<% else %>
10-
<%= link_to "☐", complete_task_path(task.id), method: :patch %>
10+
<%= link_to "☐", toggle_complete_task_path(task.id), method: :patch %>
1111
<%= link_to task.name, task_path(task.id) %>
1212
<% end %>
13-
<%= link_to 'X', delete_task_path(task.id), method: :delete, data: {comfirm: 'Are you sure you want to delete?'} %>
13+
<%= link_to 'X', delete_task_path(task.id), method: :delete, data: { comfirm: 'Are you sure you want to delete?' } %> #jqueryneeded!!? stack overflow
1414
</li>
1515
<% end %>
1616
</ul>

config/routes.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@
88
get '/tasks/:id/edit', to: 'tasks#edit', as: 'edit_task'
99
patch '/tasks/:id', to: 'tasks#update', as: 'update_task'
1010
delete '/tasks/:id/delete', to: 'tasks#delete', as: 'delete_task'
11-
patch '/tasks/:id/complete', to: 'tasks#complete', as: 'complete_task'
11+
patch '/tasks/:id/toggle_complete', to: 'tasks#toggle_complete', as: 'toggle_complete_task'
1212
end

0 commit comments

Comments
 (0)