Misc cleanup to tasks (#37)
This commit is contained in:
parent
aa02a8441b
commit
177067ab00
5 changed files with 20 additions and 50 deletions
|
@ -214,18 +214,6 @@ defmodule TaskEditComponent do
|
|||
<%= error_tag(f, :due_at) %>
|
||||
</div>
|
||||
|
||||
<div class="field">
|
||||
<%= label(f, :finished_at, class: "label") do %>
|
||||
<%= if(is_nil(@task.finished_at)) do %>
|
||||
<%= checkbox(f, :finished_at) %>
|
||||
<% else %>
|
||||
<%= checkbox(f, :finished_at, checked_value: @task.finished_at) %>
|
||||
<% end %>
|
||||
finished
|
||||
<% end %>
|
||||
<%= error_tag(f, :finished_at) %>
|
||||
</div>
|
||||
|
||||
<div class="control">
|
||||
<%= submit("save", class: "button is-dark is-small") %>
|
||||
</div>
|
||||
|
|
|
@ -12,7 +12,7 @@ defmodule PlannerWeb.TasksLive do
|
|||
{:ok, socket}
|
||||
end
|
||||
|
||||
def handle_params(%{"id" => task_id}, _, socket) do
|
||||
def handle_params(%{"task_id" => task_id}, _, socket) do
|
||||
case Tasks.verify_task_id_from_url(task_id) do
|
||||
true -> {:noreply, assign(socket, :active_task, task_id)}
|
||||
_ -> {:noreply, push_patch(socket, to: Routes.tasks_path(socket, :index))}
|
||||
|
@ -76,7 +76,7 @@ defmodule PlannerWeb.TasksLive do
|
|||
end
|
||||
|
||||
def handle_event("new-task", %{"task" => task_params}, socket) do
|
||||
case Tasks.add_task(task_params) do
|
||||
case Tasks.create_task(task_params) do
|
||||
{:ok, task} ->
|
||||
socket =
|
||||
socket
|
||||
|
|
|
@ -53,8 +53,8 @@ defmodule PlannerWeb.Router do
|
|||
pipe_through([:browser, :require_authenticated_user])
|
||||
|
||||
live("/", TasksLive, :index)
|
||||
live("/:id", TasksLive, :show)
|
||||
live("/:id/edit", TasksLive, :edit)
|
||||
live("/:task_id", TasksLive, :show)
|
||||
live("/:task_id/edit", TasksLive, :edit)
|
||||
|
||||
get("/users/settings", UserSettingsController, :edit)
|
||||
put("/users/settings/update_password", UserSettingsController, :update_password)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue