feat: in-app bounty edit & delete (#159)

Author: zcesur

lib/algora/bounties/bounties.ex

@@ -1585,4 +1585,17 @@ defmodule Algora.Bounties do
   def get_attempt_emoji(%Attempt{status: :inactive}), do: "🔴"
   def get_attempt_emoji(%Attempt{warnings_count: count}) when count > 0, do: "🟡"
   def get_attempt_emoji(%Attempt{status: :active}), do: "🟢"
+
+  @spec delete_bounty(Bounty.t()) :: {:ok, Bounty.t()} | {:error, Ecto.Changeset.t()}
+  def delete_bounty(%Bounty{} = bounty) do
+    Repo.transact(fn ->
+      with {:ok, updated_bounty} <-
+             bounty
+             |> Bounty.changeset(%{status: :cancelled})
+             |> Repo.update() do
+        broadcast()
+        {:ok, updated_bounty}
+      end
+    end)
+  end
 end

lib/algora/bounties/jobs/notify_bounty.ex

@@ -80,6 +80,7 @@ defmodule Algora.Bounties.Jobs.NotifyBounty do
            Workspace.ensure_ticket(token, ticket_ref.owner, ticket_ref.repo, ticket_ref.number),
          bounties when bounties != [] <- Bounties.list_bounties(ticket_id: ticket.id),
          {:ok, _} <- Github.add_labels(token, ticket_ref.owner, ticket_ref.repo, ticket_ref.number, ["💎 Bounty"]),
+         :ok <- Workspace.remove_existing_amount_labels(token, ticket_ref.owner, ticket_ref.repo, ticket_ref.number),
          :ok <-
            Workspace.add_amount_label(token, ticket_ref.owner, ticket_ref.repo, ticket_ref.number, Money.parse(amount)) do
       attempts = Bounties.list_attempts_for_ticket(ticket.id)

lib/algora/bounties/schemas/bounty.ex

@@ -47,7 +47,8 @@ defmodule Algora.Bounties.Bounty do
       :shared_with,
       :hours_per_week,
       :hourly_rate,
-      :contract_type
+      :contract_type,
+      :status
     ])
     |> validate_required([:amount, :ticket_id, :owner_id, :creator_id])
     |> generate_id()

lib/algora/integrations/github/behaviour.ex

@@ -24,13 +24,17 @@ defmodule Algora.Github.Behaviour do
               {:ok, map()} | {:error, String.t()}
   @callback update_issue_comment(token(), String.t(), String.t(), integer(), String.t()) ::
               {:ok, map()} | {:error, String.t()}
+  @callback delete_issue_comment(token(), String.t(), String.t(), integer()) ::
+              {:ok, map()} | {:error, String.t()}
   @callback list_user_repositories(token(), String.t(), keyword()) :: {:ok, [map()]} | {:error, String.t()}
   @callback list_repository_events(token(), String.t(), String.t(), keyword()) :: {:ok, [map()]} | {:error, String.t()}
   @callback list_repository_comments(token(), String.t(), String.t(), keyword()) :: {:ok, [map()]} | {:error, String.t()}
   @callback list_repository_languages(token(), String.t(), String.t()) :: {:ok, [map()]} | {:error, String.t()}
   @callback list_repository_contributors(token(), String.t(), String.t()) :: {:ok, [map()]} | {:error, String.t()}
   @callback add_labels(token(), String.t(), String.t(), integer(), [String.t()]) :: {:ok, [map()]} | {:error, String.t()}
+  @callback list_labels(token(), String.t(), String.t(), integer()) :: {:ok, [map()]} | {:error, String.t()}
   @callback create_label(token(), String.t(), String.t(), map()) :: {:ok, map()} | {:error, String.t()}
   @callback get_label(token(), String.t(), String.t(), String.t()) :: {:ok, map()} | {:error, String.t()}
   @callback remove_label(token(), String.t(), String.t(), String.t()) :: {:ok, map()} | {:error, String.t()}
+  @callback remove_label_from_issue(token(), String.t(), String.t(), integer(), String.t()) :: {:ok, map()} | {:error, String.t()}
 end

lib/algora/integrations/github/client.ex

@@ -241,6 +241,22 @@ defmodule Algora.Github.Client do
     )
   end
 
+  @impl true
+  def delete_issue_comment(access_token, owner, repo, comment_id) do
+    access_token
+    |> fetch(
+      "/repos/#{owner}/#{repo}/issues/comments/#{comment_id}",
+      "DELETE"
+    )
+    |> case do
+      {:error, %Jason.DecodeError{position: 0, token: nil, data: ""}} ->
+        {:ok, nil}
+
+      res ->
+        res
+    end
+  end
+
   @impl true
   def list_user_repositories(access_token, username, opts \\ []) do
     fetch(access_token, "/users/#{username}/repos#{build_query(opts)}")
@@ -273,6 +289,11 @@ defmodule Algora.Github.Client do
     })
   end
 
+  @impl true
+  def list_labels(access_token, owner, repo, number) do
+    fetch(access_token, "/repos/#{owner}/#{repo}/issues/#{number}/labels")
+  end
+
   @impl true
   def create_label(access_token, owner, repo, label) do
     fetch(access_token, "/repos/#{owner}/#{repo}/labels", "POST", label)
@@ -285,6 +306,27 @@ defmodule Algora.Github.Client do
 
   @impl true
   def remove_label(access_token, owner, repo, label) do
-    fetch(access_token, "/repos/#{owner}/#{repo}/labels/#{label}", "DELETE")
+    access_token
+    |> fetch("/repos/#{owner}/#{repo}/labels/#{label}", "DELETE")
+    |> case do
+      {:error, %Jason.DecodeError{position: 0, token: nil, data: ""}} ->
+        {:ok, nil}
+
+      res ->
+        res
+    end
+  end
+
+  @impl true
+  def remove_label_from_issue(access_token, owner, repo, number, label) do
+    access_token
+    |> fetch("/repos/#{owner}/#{repo}/issues/#{number}/labels/#{URI.encode(label)}", "DELETE")
+    |> case do
+      {:error, %Jason.DecodeError{position: 0, token: nil, data: ""}} ->
+        {:ok, nil}
+
+      res ->
+        res
+    end
   end
 end

lib/algora/integrations/github/github.ex

@@ -126,6 +126,10 @@ defmodule Algora.Github do
   def update_issue_comment(token, owner, repo, comment_id, body),
     do: client().update_issue_comment(token, owner, repo, comment_id, body)
 
+  @impl true
+  def delete_issue_comment(token, owner, repo, comment_id),
+    do: client().delete_issue_comment(token, owner, repo, comment_id)
+
   @impl true
   def list_user_repositories(token, username, opts \\ []), do: client().list_user_repositories(token, username, opts)
 
@@ -145,6 +149,9 @@ defmodule Algora.Github do
   @impl true
   def add_labels(token, owner, repo, number, labels), do: client().add_labels(token, owner, repo, number, labels)
 
+  @impl true
+  def list_labels(token, owner, repo, number), do: client().list_labels(token, owner, repo, number)
+
   @impl true
   def create_label(token, owner, repo, label), do: client().create_label(token, owner, repo, label)
 
@@ -153,4 +160,7 @@ defmodule Algora.Github do
 
   @impl true
   def remove_label(token, owner, repo, label), do: client().remove_label(token, owner, repo, label)
+
+  @impl true
+  def remove_label_from_issue(token, owner, repo, number, label), do: client().remove_label_from_issue(token, owner, repo, number, label)
 end

lib/algora/workspace/workspace.ex

@@ -882,4 +882,30 @@ defmodule Algora.Workspace do
       end)
     end)
   end
+
+  def remove_existing_amount_labels(token, owner, repo, number) do
+    case Github.list_labels(token, owner, repo, number) do
+      {:ok, labels} ->
+        amount_labels =
+          labels
+          |> Enum.filter(fn label -> String.starts_with?(label["name"], "$") end)
+          |> Enum.map(fn label -> label["name"] end)
+
+        case amount_labels do
+          [] ->
+            :ok
+
+          _ ->
+            Enum.each(amount_labels, fn label_name ->
+              Github.remove_label_from_issue(token, owner, repo, number, label_name)
+            end)
+
+            :ok
+        end
+
+      {:error, reason} ->
+        Logger.warning("Failed to list labels for #{owner}/#{repo}##{number}: #{inspect(reason)}")
+        :ok
+    end
+  end
 end