feat: add repository navigation and enhance BountyLive functionality

zcesur · zcesur · commit 1d19562248c4 · 2025-04-01T13:53:01.000+03:00
- Introduced a new route for repository-specific live sessions in the router.
- Added a RepoNav component for improved navigation within repository contexts.
- Enhanced BountyLive to support dynamic loading of bounties based on repository owner and name.
- Updated ticket reference handling to streamline data assignment in BountyLive.
- Refactored the mount function in BountyLive to incorporate repository parameters for better context management.
diff --git a/lib/algora_web/live/bounty_live.ex b/lib/algora_web/live/bounty_live.ex
@@ -3,6 +3,7 @@ defmodule AlgoraWeb.BountyLive do
   use AlgoraWeb, :live_view
 
   import Ecto.Changeset
+  import Ecto.Query
 
   alias Algora.Accounts
   alias Algora.Admin
@@ -65,13 +66,49 @@ defmodule AlgoraWeb.BountyLive do
     bounty =
       Bounty
       |> Repo.get!(bounty_id)
-      |> Repo.preload([
-        :owner,
-        :creator,
-        :transactions,
-        ticket: [repository: [:user]]
-      ])
+      |> Repo.preload([:owner, :creator, :transactions, ticket: [repository: [:user]]])
 
+    ticket_ref = %{
+      owner: bounty.ticket.repository.user.provider_login,
+      repo: bounty.ticket.repository.name,
+      number: bounty.ticket.number
+    }
+
+    socket
+    |> assign(:bounty, bounty)
+    |> assign(:ticket_ref, ticket_ref)
+    |> on_mount(bounty)
+  end
+
+  @impl true
+  def mount(%{"repo_owner" => repo_owner, "repo_name" => repo_name, "number" => number}, _session, socket) do
+    number = String.to_integer(number)
+
+    ticket_ref = %{owner: repo_owner, repo: repo_name, number: number}
+
+    bounty =
+      from(b in Bounty,
+        join: t in assoc(b, :ticket),
+        join: r in assoc(t, :repository),
+        join: u in assoc(r, :user),
+        where: u.provider == "github",
+        where: u.provider_login == ^repo_owner,
+        where: r.name == ^repo_name,
+        where: t.number == ^number,
+        # TODO: pool bounties
+        limit: 1,
+        order_by: [asc: b.inserted_at]
+      )
+      |> Repo.one()
+      |> Repo.preload([:owner, :creator, :transactions, ticket: [repository: [:user]]])
+
+    socket
+    |> assign(:bounty, bounty)
+    |> assign(:ticket_ref, ticket_ref)
+    |> on_mount(bounty)
+  end
+
+  defp on_mount(socket, bounty) do
     debits = Enum.filter(bounty.transactions, &(&1.type == :debit and &1.status == :succeeded))
 
     total_paid =
@@ -81,8 +118,6 @@ defmodule AlgoraWeb.BountyLive do
 
     ticket_body_html = Algora.Markdown.render(bounty.ticket.description)
 
-    contexts = contexts(bounty)
-
     reward_changeset =
       RewardBountyForm.changeset(%RewardBountyForm{}, %{
         tip_percentage: 0,
@@ -102,11 +137,9 @@ defmodule AlgoraWeb.BountyLive do
     {:ok,
      socket
      |> assign(:page_title, bounty.ticket.title)
-     |> assign(:bounty, bounty)
      |> assign(:ticket, bounty.ticket)
      |> assign(:total_paid, total_paid)
      |> assign(:ticket_body_html, ticket_body_html)
-     |> assign(:contexts, contexts)
      |> assign(:show_reward_modal, false)
      |> assign(:show_exclusive_modal, false)
      |> assign(:selected_context, nil)
@@ -126,11 +159,6 @@ defmodule AlgoraWeb.BountyLive do
     {:noreply, socket}
   end
 
-  # @impl true
-  # def handle_params(%{"context" => context_id}, _url, socket) do
-  #   {:noreply, socket |> assign_selected_context(context_id) |> assign_line_items(nil)}
-  # end
-
   @impl true
   def handle_params(_params, _url, socket) do
     {:noreply, socket}
@@ -164,7 +192,7 @@ defmodule AlgoraWeb.BountyLive do
 
     {:noreply,
      socket
-     |> update(:messages, &(&1 ++ [message]))
+     |> Phoenix.Component.update(:messages, &(&1 ++ [message]))
      |> push_event("clear-input", %{selector: "#message-input"})}
   end
 
@@ -367,22 +395,38 @@ defmodule AlgoraWeb.BountyLive do
                       <.social_share_button
                         id="twitter-share-url"
                         icon="tabler-brand-x"
-                        value={url(~p"/org/#{@bounty.owner.handle}/bounties/#{@bounty.id}")}
+                        value={
+                          url(
+                            ~p"/#{@ticket_ref.owner}/#{@ticket_ref.repo}/issues/#{@ticket_ref.number}"
+                          )
+                        }
                       />
                       <.social_share_button
                         id="reddit-share-url"
                         icon="tabler-brand-reddit"
-                        value={url(~p"/org/#{@bounty.owner.handle}/bounties/#{@bounty.id}")}
+                        value={
+                          url(
+                            ~p"/#{@ticket_ref.owner}/#{@ticket_ref.repo}/issues/#{@ticket_ref.number}"
+                          )
+                        }
                       />
                       <.social_share_button
                         id="linkedin-share-url"
                         icon="tabler-brand-linkedin"
-                        value={url(~p"/org/#{@bounty.owner.handle}/bounties/#{@bounty.id}")}
+                        value={
+                          url(
+                            ~p"/#{@ticket_ref.owner}/#{@ticket_ref.repo}/issues/#{@ticket_ref.number}"
+                          )
+                        }
                       />
                       <.social_share_button
                         id="hackernews-share-url"
                         icon="tabler-brand-ycombinator"
-                        value={url(~p"/org/#{@bounty.owner.handle}/bounties/#{@bounty.id}")}
+                        value={
+                          url(
+                            ~p"/#{@ticket_ref.owner}/#{@ticket_ref.repo}/issues/#{@ticket_ref.number}"
+                          )
+                        }
                       />
                     </div>
                   </div>
@@ -635,16 +679,6 @@ defmodule AlgoraWeb.BountyLive do
     """
   end
 
-  # defp assign_selected_context(socket, context_id) do
-  #   case Enum.find(socket.assigns.contexts, &(&1.id == context_id)) do
-  #     nil ->
-  #       socket
-
-  #     context ->
-  #       assign(socket, :selected_context, context)
-  #   end
-  # end
-
   defp assign_recipient(socket, github_handle) do
     case Workspace.ensure_user(Admin.token!(), github_handle) do
       {:ok, recipient} ->
@@ -658,7 +692,7 @@ defmodule AlgoraWeb.BountyLive do
   defp assign_line_items(socket) do
     amount = calculate_final_amount(socket.assigns.reward_form.source)
     recipient = socket.assigns.recipient
-    ticket_ref = ticket_ref(socket)
+    ticket_ref = socket.assigns.ticket_ref
 
     line_items =
       if recipient do
@@ -685,20 +719,12 @@ defmodule AlgoraWeb.BountyLive do
     assign(socket, :line_items, line_items)
   end
 
-  defp ticket_ref(socket) do
-    %{
-      owner: socket.assigns.ticket.repository.user.provider_login,
-      repo: socket.assigns.ticket.repository.name,
-      number: socket.assigns.ticket.number
-    }
-  end
-
   defp reward_bounty(socket, bounty, changeset) do
     final_amount = calculate_final_amount(changeset)
 
     Bounties.reward_bounty(
       %{owner: bounty.owner, amount: final_amount, bounty_id: bounty.id, claims: []},
-      ticket_ref: ticket_ref(socket),
+      ticket_ref: socket.assigns.ticket_ref,
       recipient: socket.assigns.recipient
     )
   end
@@ -745,10 +771,6 @@ defmodule AlgoraWeb.BountyLive do
     """
   end
 
-  defp contexts(_bounty) do
-    Accounts.list_featured_developers()
-  end
-
   defp close_drawers(socket) do
     socket
     |> assign(:show_reward_modal, false)
diff --git a/lib/algora_web/live/org/repo_nav.ex b/lib/algora_web/live/org/repo_nav.ex
@@ -0,0 +1,107 @@
+defmodule AlgoraWeb.Org.RepoNav do
+  @moduledoc false
+  use Phoenix.Component
+
+  import Phoenix.LiveView
+
+  alias Algora.Organizations
+  alias AlgoraWeb.OrgAuth
+
+  def on_mount(:default, %{"repo_owner" => repo_owner} = params, _session, socket) do
+    current_user = socket.assigns[:current_user]
+    current_org = Organizations.get_org_by(provider_login: repo_owner, provider: "github")
+    current_user_role = OrgAuth.get_user_role(current_user, current_org)
+
+    {:cont,
+     socket
+     |> assign(:screenshot?, not is_nil(params["screenshot"]))
+     |> assign(:new_bounty_form, to_form(%{"github_issue_url" => "", "amount" => ""}))
+     |> assign(:current_org, current_org)
+     |> assign(:current_user_role, current_user_role)
+     |> assign(:nav, nav_items(current_org.handle, current_user_role))
+     |> assign(:contacts, [])
+     |> attach_hook(:active_tab, :handle_params, &handle_active_tab_params/3)}
+  end
+
+  defp handle_active_tab_params(_params, _url, socket) do
+    active_tab =
+      case {socket.view, socket.assigns.live_action} do
+        {AlgoraWeb.Org.DashboardLive, _} -> :dashboard
+        {AlgoraWeb.Org.HomeLive, _} -> :home
+        {AlgoraWeb.Org.BountiesLive, _} -> :bounties
+        {AlgoraWeb.Org.ProjectsLive, _} -> :projects
+        {AlgoraWeb.Project.ViewLive, _} -> :projects
+        {AlgoraWeb.Org.SettingsLive, _} -> :settings
+        {AlgoraWeb.Org.MembersLive, _} -> :members
+        {_, _} -> nil
+      end
+
+    {:cont, assign(socket, :active_tab, active_tab)}
+  end
+
+  def nav_items(org_handle, current_user_role) do
+    [
+      %{
+        title: "Overview",
+        items: build_nav_items(org_handle, current_user_role)
+      }
+    ]
+  end
+
+  defp build_nav_items(org_handle, current_user_role) do
+    Enum.filter(
+      [
+        %{
+          href: "/org/#{org_handle}",
+          tab: :dashboard,
+          icon: "tabler-sparkles",
+          label: "Dashboard",
+          roles: [:admin, :mod]
+        },
+        %{
+          href: "/org/#{org_handle}/home",
+          tab: :home,
+          icon: "tabler-home",
+          label: "Home",
+          roles: [:admin, :mod, :expert, :none]
+        },
+        %{
+          href: "/org/#{org_handle}/bounties",
+          tab: :bounties,
+          icon: "tabler-diamond",
+          label: "Bounties",
+          roles: [:admin, :mod, :expert, :none]
+        },
+        %{
+          href: "/org/#{org_handle}/leaderboard",
+          tab: :leaderboard,
+          icon: "tabler-trophy",
+          label: "Leaderboard",
+          roles: [:admin, :mod, :expert, :none]
+        },
+        %{
+          href: "/org/#{org_handle}/team",
+          tab: :team,
+          icon: "tabler-users",
+          label: "Team",
+          roles: [:admin, :mod, :expert, :none]
+        },
+        %{
+          href: "/org/#{org_handle}/transactions",
+          tab: :transactions,
+          icon: "tabler-credit-card",
+          label: "Transactions",
+          roles: [:admin]
+        },
+        %{
+          href: "/org/#{org_handle}/settings",
+          tab: :settings,
+          icon: "tabler-settings",
+          label: "Settings",
+          roles: [:admin]
+        }
+      ],
+      fn item -> current_user_role in item[:roles] end
+    )
+  end
+end
diff --git a/lib/algora_web/router.ex b/lib/algora_web/router.ex
@@ -88,18 +88,26 @@ defmodule AlgoraWeb.Router do
       end
     end
 
+    scope "/:repo_owner/:repo_name" do
+      live_session :repo,
+        layout: {AlgoraWeb.Layouts, :user},
+        on_mount: [{AlgoraWeb.UserAuth, :current_user}, AlgoraWeb.Org.RepoNav] do
+        live "/issues/:number", BountyLive
+        live "/pull/:number", BountyLive
+      end
+    end
+
     scope "/org/:org_handle" do
       live_session :org,
         layout: {AlgoraWeb.Layouts, :user},
         on_mount: [{AlgoraWeb.UserAuth, :current_user}, AlgoraWeb.Org.Nav] do
         live "/", Org.DashboardLive, :index
         live "/home", Org.HomeLive, :index
         live "/bounties", Org.BountiesLive, :index
+        live "/bounties/:id", BountyLive, :index
         live "/contracts/:id", Contract.ViewLive
         live "/team", Org.TeamLive, :index
         live "/leaderboard", Org.LeaderboardLive, :index
-        # TODO: allow access to invited users
-        live "/bounties/:id", BountyLive
       end
 
       live_session :org_admin,
