show: regroup find_step helper out of handle_event clauses
e5029ee · gmorell · 2026-06-25 20:03
Files changed
modified
lib/git_gud_web/live/workflow_live/show.ex
+4
−4
@@ -51,6 +51,10 @@ defmodule GitGudWeb.WorkflowLive.Show do
| 51 | 51 | |
| 52 | 52 | defp stream_name(step_id), do: :"step_log_#{step_id}" |
| 53 | 53 | |
| 54 | + defp find_step(run, id) do | |
| 55 | + Enum.find_value(run.jobs, fn job -> Enum.find(job.steps, &(&1.id == id)) end) | |
| 56 | + end | |
| 57 | + | |
| 54 | 58 | @impl true |
| 55 | 59 | def handle_info({:workflow_log, step_id, seq, body}, socket) do |
| 56 | 60 | if has_step?(socket, step_id) do |
@@ -114,10 +118,6 @@ defmodule GitGudWeb.WorkflowLive.Show do
| 114 | 118 | end |
| 115 | 119 | end |
| 116 | 120 | |
| 117 | − defp find_step(run, id) do | |
| 118 | − Enum.find_value(run.jobs, fn job -> Enum.find(job.steps, &(&1.id == id)) end) | |
| 119 | − end | |
| 120 | − | |
| 121 | 121 | def handle_event("rerun", _params, socket) do |
| 122 | 122 | cond do |
| 123 | 123 | not socket.assigns.can_admin? -> |
Parents: 268bd5d