Levi's solution #25

levikl · 2018-01-23T05:38:32Z

jaybobo · 2018-01-25T14:57:36Z

lklingler-rails-tasksapp/app/controllers/tasks_controller.rb

+        format.html { redirect_to tasks_path, notice: 'Task was successfully created.' }
+        format.json { render :show, status: :created, location: @task }
+      else
+        format.html { render :new }


There's no issue here but the rails generators will give you more than what you'll need. Sometimes, it's better to write your controllers from scratch.

jaybobo · 2018-01-25T14:58:07Z

lklingler-rails-tasksapp/app/views/tasks/index.html.erb

+    <% if task.completed %>
+      <li><%= link_to "[x] #{task.title}", task %></li>
+    <% else %>
+      <li><%= link_to "[ &nbsp;] #{task.title}".html_safe, task %></li>


Keep in mind the danger of using .html_safe

https://product.reverb.com/stay-safe-while-using-html-safe-in-rails-9e368836fac1

Levi Klingler added 2 commits January 23, 2018 00:37

Reimplemented mock CRUD app using Rails 5

83d7356

Ordered tasks on index page by id

9f6a3d5

jaybobo reviewed Jan 25, 2018

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Levi's solution #25

Levi's solution #25

levikl commented Jan 23, 2018

jaybobo Jan 25, 2018

jaybobo Jan 25, 2018

Levi's solution #25

Are you sure you want to change the base?

Levi's solution #25

Conversation

levikl commented Jan 23, 2018

jaybobo Jan 25, 2018

Choose a reason for hiding this comment

jaybobo Jan 25, 2018

Choose a reason for hiding this comment