run migrations via entrypoint script

Dockerfile

@@ -90,4 +90,4 @@ USER nobody
 # above and adding an entrypoint. See https://github.com/krallin/tini for details
 # ENTRYPOINT ["/tini", "--"]
 
-CMD ["/app/bin/server"]
+ENTRYPOINT ["/app/bin/docker-entrypoint.sh"]

README.md

@@ -255,7 +255,7 @@ For testing the production Docker build locally:
    docker compose -f docker-compose.prod.yml up
    ```
 
-5. **Run database migrations:**
+5. **Database migrations run automatically** on app start. For manual migration:
    ```bash
    docker compose -f docker-compose.prod.yml exec app /app/bin/mv eval "Mv.Release.migrate"
    ```

lib/mv_web/components/core_components.ex

@@ -474,7 +474,6 @@ defmodule MvWeb.CoreComponents do
 
   slot :col, required: true do
     attr :label, :string
-    attr :col_click, :any, doc: "optional column-specific click handler that overrides row_click"
   end
 
   slot :action, doc: "the slot for showing user actions in the last table column"
@@ -510,11 +509,8 @@ defmodule MvWeb.CoreComponents do
           <tr :for={row <- @rows} id={@row_id && @row_id.(row)}>
             <td
               :for={col <- @col}
-              phx-click={
-                (col[:col_click] && col[:col_click].(@row_item.(row))) ||
-                  (@row_click && @row_click.(row))
-              }
-              class={["max-w-xs truncate", (col[:col_click] || @row_click) && "hover:cursor-pointer"]}
+              phx-click={@row_click && @row_click.(row)}
+              class={["max-w-xs truncate", @row_click && "hover:cursor-pointer"]}
             >
               {render_slot(col, @row_item.(row))}
             </td>

lib/mv_web/live/member_live/index.ex

@@ -1082,16 +1082,6 @@ defmodule MvWeb.MemberLive.Index do
     |> Enum.map(&format_member_email/1)
   end
 
-  @doc """
-  Returns a JS command to toggle member selection when clicking the checkbox column.
-
-  Used as `col_click` handler to ensure clicking anywhere in the checkbox column
-  toggles the checkbox instead of navigating to the member details.
-  """
-  def checkbox_column_click(member) do
-    JS.push("select_member", value: %{id: member.id})
-  end
-
   # Formats a member's email in the format "First Last <email>"
   # Used for copy_emails feature and mailto links to create email-client-friendly format.
   def format_member_email(member) do

lib/mv_web/live/member_live/index.html.heex

@@ -65,7 +65,6 @@
 <!-- <:col :let={member} label="Id">{member.id}</:col> -->
     <:col
       :let={member}
-      col_click={&MvWeb.MemberLive.Index.checkbox_column_click/1}
       label={
         ~H"""
         <.input
@@ -82,7 +81,11 @@
       <.input
         type="checkbox"
         name={member.id}
+        phx-click="select_member"
+        phx-value-id={member.id}
         checked={MapSet.member?(@selected_members, member.id)}
+        phx-capture-click
+        phx-stop-propagation
         aria-label={gettext("Select member")}
         role="checkbox"
       />

rel/overlays/bin/docker-entrypoint.sh

@@ -0,0 +1,9 @@
+#!/bin/sh
+set -e
+
+echo "==> Running database migrations..."
+/app/bin/migrate
+
+echo "==> Starting application..."
+exec /app/bin/server
+

test/mv_web/member_live/index_test.exs

@@ -285,9 +285,14 @@ defmodule MvWeb.MemberLive.IndexTest do
       conn = conn_with_oidc_user(conn)
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select two members by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => member1.id})
-      render_click(view, "select_member", %{"id" => member2.id})
+      # Select two members
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member1.id}']")
+      |> render_click()
+
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member2.id}']")
+      |> render_click()
 
       # Trigger copy_emails event
       view |> element("#copy-emails-btn") |> render_click()
@@ -331,8 +336,10 @@ defmodule MvWeb.MemberLive.IndexTest do
       conn = conn_with_oidc_user(conn)
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select member with umlauts by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => member3.id})
+      # Select member with umlauts
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member3.id}']")
+      |> render_click()
 
       # Trigger copy_emails event - should not crash
       view |> element("#copy-emails-btn") |> render_click()
@@ -348,8 +355,10 @@ defmodule MvWeb.MemberLive.IndexTest do
       conn = conn_with_oidc_user(conn)
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select a member by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => member1.id})
+      # Select a member
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member1.id}']")
+      |> render_click()
 
       # Delete the member from the database
       Ash.destroy!(member1)
@@ -370,9 +379,14 @@ defmodule MvWeb.MemberLive.IndexTest do
       conn = conn_with_oidc_user(conn)
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select two members by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => member1.id})
-      render_click(view, "select_member", %{"id" => member2.id})
+      # Select two members
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member1.id}']")
+      |> render_click()
+
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member2.id}']")
+      |> render_click()
 
       # Get the socket state to verify the formatted email string
       state = :sys.get_state(view.pid)
@@ -401,8 +415,10 @@ defmodule MvWeb.MemberLive.IndexTest do
 
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select the test member by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => test_member.id})
+      # Select the test member
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{test_member.id}']")
+      |> render_click()
 
       # The format should be "Test Format <test.format@example.com>"
       # We verify this by checking the flash shows 1 email was copied
@@ -425,8 +441,10 @@ defmodule MvWeb.MemberLive.IndexTest do
       conn = conn_with_oidc_user(conn)
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select a member by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => member1.id})
+      # Select a member
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member1.id}']")
+      |> render_click()
 
       # Button should now be visible
       assert has_element?(view, "#copy-emails-btn")
@@ -439,8 +457,10 @@ defmodule MvWeb.MemberLive.IndexTest do
       conn = conn_with_oidc_user(conn)
       {:ok, view, _html} = live(conn, "/members")
 
-      # Select a member by sending the select_member event directly
-      render_click(view, "select_member", %{"id" => member1.id})
+      # Select a member
+      view
+      |> element("[phx-click='select_member'][phx-value-id='#{member1.id}']")
+      |> render_click()
 
       # Click copy button
       view |> element("#copy-emails-btn") |> render_click()