Show cron name in header when it differs

Author: sorentwo

lib/oban/web/live/crons/detail_component.ex

@@ -1,7 +1,7 @@
 defmodule Oban.Web.Crons.DetailComponent do
   use Oban.Web, :live_component
 
-  import Oban.Web.Crons.Helpers, only: [state_icon: 1, maybe_to_unix: 1]
+  import Oban.Web.Crons.Helpers, only: [state_icon: 1, maybe_to_unix: 1, show_name?: 1]
   import Oban.Web.FormComponents
 
   alias Oban.Job
@@ -23,7 +23,12 @@ defmodule Oban.Web.Crons.DetailComponent do
           phx-hook="Tippy"
         >
           <Icons.arrow_left class="w-5 h-5" />
-          <span class="text-lg font-bold ml-2">{@cron.worker}</span>
+          <span class="text-lg font-bold ml-2">
+            {@cron.worker}
+            <span :if={show_name?(@cron)} class="font-normal text-gray-500 dark:text-gray-400">
+              ({@cron.name})
+            </span>
+          </span>
         </.link>
 
         <div class="flex space-x-3">

lib/oban/web/live/crons/helpers.ex

@@ -47,4 +47,7 @@ defmodule Oban.Web.Crons.Helpers do
     |> DateTime.from_naive!("Etc/UTC")
     |> DateTime.to_unix(:millisecond)
   end
+
+  def show_name?(%{dynamic?: true, name: name, worker: worker}), do: name != worker
+  def show_name?(_cron), do: false
 end

lib/oban/web/live/queues/detail_instance_component.ex

@@ -203,5 +203,4 @@ defmodule Oban.Web.Queues.DetailInstanceComponent do
 
     {:noreply, assign(socket, local_limit: limit, editing?: false)}
   end
-
 end

lib/oban/web/pages/crons_page.ex

@@ -3,7 +3,7 @@ defmodule Oban.Web.CronsPage do
 
   use Oban.Web, :live_component
 
-  import Oban.Web.Crons.Helpers, only: [maybe_to_unix: 1, state_icon: 1]
+  import Oban.Web.Crons.Helpers, only: [maybe_to_unix: 1, show_name?: 1, state_icon: 1]
 
   alias Oban.Web.Crons.{DetailComponent, NewComponent}
   alias Oban.Web.{Cron, CronQuery, Page, QueueQuery, SearchComponent, SortComponent, Utils}
@@ -503,8 +503,4 @@ defmodule Oban.Web.CronsPage do
   defp has_tags?(opts), do: Map.has_key?(opts, "tags") and opts["tags"] != []
 
   defp get_tags(opts), do: Map.get(opts, "tags", [])
-
-  defp show_name?(%{dynamic?: true, name: name, worker: worker}), do: name != worker
-
-  defp show_name?(_cron), do: false
 end

test/oban/web/pages/queues/detail_test.exs

@@ -108,7 +108,10 @@ defmodule Oban.Web.Pages.Queues.DetailTest do
 
     live
     |> form("#global-form")
-    |> render_submit(%{global_partition_fields: "meta,worker", global_partition_keys: "tenant_id"})
+    |> render_submit(%{
+      global_partition_fields: "meta,worker",
+      global_partition_keys: "tenant_id"
+    })
 
     assert_signal(%{
       "action" => "scale",