feat: update fees

Author: zcesur

lib/algora/accounts/schemas/user.ex

@@ -39,6 +39,7 @@ defmodule Algora.Accounts.User do
     field :featured, :boolean, default: false
     field :priority, :integer, default: 0
     field :fee_pct, :integer, default: 9
+    field :fee_pct_prev, :integer, default: 9
     field :seeded, :boolean, default: false
     field :activated, :boolean, default: false
     field :max_open_attempts, :integer, default: 3

lib/algora/bounties/bounties.ex

@@ -766,17 +766,17 @@ defmodule Algora.Bounties do
           %{
             owner: User.t(),
             amount: Money.t(),
-            bounty_id: String.t(),
+            bounty: Bounty.t(),
             claims: [Claim.t()]
           },
           opts :: [ticket_ref: %{owner: String.t(), repo: String.t(), number: integer()}, recipient: User.t()]
         ) ::
           {:ok, String.t()} | {:error, atom()}
-  def reward_bounty(%{owner: owner, amount: amount, bounty_id: bounty_id, claims: claims}, opts \\ []) do
+  def reward_bounty(%{owner: owner, amount: amount, bounty: bounty, claims: claims}, opts \\ []) do
     create_payment_session(
       %{owner: owner, amount: amount, description: "Bounty payment for OSS contributions"},
       ticket_ref: opts[:ticket_ref],
-      bounty_id: bounty_id,
+      bounty: bounty,
       claims: claims,
       recipient: opts[:recipient]
     )
@@ -785,20 +785,28 @@ defmodule Algora.Bounties do
   @spec generate_line_items(
           %{owner: User.t(), amount: Money.t()},
           opts :: [
+            bounty: Bounty.t(),
             ticket_ref: %{owner: String.t(), repo: String.t(), number: integer()},
             claims: [Claim.t()],
             recipient: User.t()
           ]
         ) ::
           [LineItem.t()]
   def generate_line_items(%{owner: owner, amount: amount}, opts \\ []) do
+    bounty = opts[:bounty]
     ticket_ref = opts[:ticket_ref]
     recipient = opts[:recipient]
     claims = opts[:claims] || []
 
     description = if(ticket_ref, do: "#{ticket_ref[:repo]}##{ticket_ref[:number]}")
 
-    platform_fee_pct = Decimal.div(owner.fee_pct, 100)
+    platform_fee_pct =
+      if bounty && Date.before?(bounty.inserted_at, ~D[2025-04-16]) do
+        Decimal.div(owner.fee_pct_prev, 100)
+      else
+        Decimal.div(owner.fee_pct, 100)
+      end
+
     transaction_fee_pct = Payments.get_transaction_fee_pct()
 
     payouts =
@@ -845,7 +853,7 @@ defmodule Algora.Bounties do
           opts :: [
             ticket_ref: %{owner: String.t(), repo: String.t(), number: integer()},
             tip_id: String.t(),
-            bounty_id: String.t(),
+            bounty: Bounty.t(),
             claims: [Claim.t()],
             recipient: User.t()
           ]
@@ -858,17 +866,20 @@ defmodule Algora.Bounties do
       generate_line_items(%{owner: owner, amount: amount},
         ticket_ref: opts[:ticket_ref],
         recipient: opts[:recipient],
-        claims: opts[:claims]
+        claims: opts[:claims],
+        bounty: opts[:bounty]
       )
 
     gross_amount = LineItem.gross_amount(line_items)
 
+    bounty_id = if bounty = opts[:bounty], do: bounty.id
+
     Repo.transact(fn ->
       with {:ok, _charge} <-
              initialize_charge(%{
                id: Nanoid.generate(),
                user_id: owner.id,
-               bounty_id: opts[:bounty_id],
+               bounty_id: bounty_id,
                gross_amount: gross_amount,
                net_amount: amount,
                total_fee: Money.sub!(gross_amount, amount),
@@ -881,7 +892,7 @@ defmodule Algora.Bounties do
                claims: opts[:claims] || [],
                tip_id: opts[:tip_id],
                recipient_id: if(opts[:recipient], do: opts[:recipient].id),
-               bounty_id: opts[:bounty_id],
+               bounty_id: bounty_id,
                claim_id: nil,
                amount: amount,
                creator_id: owner.id,
@@ -902,7 +913,7 @@ defmodule Algora.Bounties do
           opts :: [
             ticket_ref: %{owner: String.t(), repo: String.t(), number: integer()},
             tip_id: String.t(),
-            bounty_id: String.t(),
+            bounty: Bounty.t(),
             claims: [Claim.t()],
             recipient: User.t()
           ]
@@ -915,11 +926,14 @@ defmodule Algora.Bounties do
       generate_line_items(%{owner: owner, amount: amount},
         ticket_ref: opts[:ticket_ref],
         recipient: opts[:recipient],
-        claims: opts[:claims]
+        claims: opts[:claims],
+        bounty: opts[:bounty]
       )
 
     gross_amount = LineItem.gross_amount(line_items)
 
+    bounty_id = if bounty = opts[:bounty], do: bounty.id
+
     Repo.transact(fn ->
       with {:ok, _charge} <-
              initialize_charge(%{
@@ -937,7 +951,7 @@ defmodule Algora.Bounties do
                claims: opts[:claims] || [],
                tip_id: opts[:tip_id],
                recipient_id: if(opts[:recipient], do: opts[:recipient].id),
-               bounty_id: opts[:bounty_id],
+               bounty_id: bounty_id,
                claim_id: nil,
                amount: amount,
                creator_id: owner.id,

lib/algora_web/controllers/webhooks/github_controller.ex

@@ -216,7 +216,7 @@ defmodule AlgoraWeb.Webhooks.GithubController do
                        repo: payload["repository"]["name"],
                        number: payload["pull_request"]["number"]
                      },
-                     bounty_id: autopayable_bounty.id,
+                     bounty: autopayable_bounty,
                      claims: claims
                    ),
                  {:ok, _invoice} <-

lib/algora_web/live/bounty_live.ex

@@ -747,6 +747,7 @@ defmodule AlgoraWeb.BountyLive do
             owner: socket.assigns.bounty.owner,
             amount: amount
           },
+          bounty: socket.assigns.bounty,
           ticket_ref: ticket_ref,
           recipient: recipient
         )
@@ -758,7 +759,7 @@ defmodule AlgoraWeb.BountyLive do
     final_amount = calculate_final_amount(changeset)
 
     Bounties.reward_bounty(
-      %{owner: bounty.owner, amount: final_amount, bounty_id: bounty.id, claims: []},
+      %{owner: bounty.owner, amount: final_amount, bounty: bounty, claims: []},
       ticket_ref: socket.assigns.ticket_ref,
       recipient: socket.assigns.recipient
     )

lib/algora_web/live/claim_live.ex

@@ -254,12 +254,15 @@ defmodule AlgoraWeb.ClaimLive do
   end
 
   defp assign_line_items(socket) do
+    bounty = Enum.find(socket.assigns.available_bounties, &(&1.owner_id == socket.assigns.selected_context.id))
+
     line_items =
       Bounties.generate_line_items(
         %{
           owner: socket.assigns.selected_context,
           amount: calculate_final_amount(socket.assigns.reward_bounty_form.source)
         },
+        bounty: bounty,
         ticket_ref: ticket_ref(socket),
         claims: socket.assigns.claims
       )
@@ -299,7 +302,7 @@ defmodule AlgoraWeb.ClaimLive do
       %{
         owner: socket.assigns.selected_context,
         amount: final_amount,
-        bounty_id: bounty.id,
+        bounty: bounty,
         claims: socket.assigns.claims
       },
       ticket_ref: ticket_ref(socket)

lib/algora_web/live/contract_live.ex

@@ -576,6 +576,7 @@ defmodule AlgoraWeb.ContractLive do
           owner: socket.assigns.bounty.owner,
           amount: calculate_final_amount(socket.assigns.reward_form.source)
         },
+        bounty: socket.assigns.bounty,
         ticket_ref: socket.assigns.ticket_ref,
         recipient: socket.assigns.contractor
       )
@@ -587,7 +588,7 @@ defmodule AlgoraWeb.ContractLive do
     final_amount = calculate_final_amount(changeset)
 
     Bounties.reward_bounty(
-      %{owner: bounty.owner, amount: final_amount, bounty_id: bounty.id, claims: []},
+      %{owner: bounty.owner, amount: final_amount, bounty: bounty, claims: []},
       ticket_ref: socket.assigns.ticket_ref,
       recipient: socket.assigns.contractor
     )

priv/repo/migrations/20250416085834_update_user_fee_pct.exs

@@ -0,0 +1,15 @@
+defmodule Algora.Repo.Migrations.UpdateUserFeePct do
+  use Ecto.Migration
+
+  def change do
+    alter table(:users) do
+      add :fee_pct_prev, :integer, default: 9
+    end
+
+    execute """
+    UPDATE users
+    SET fee_pct_prev = fee_pct,
+        fee_pct = 9
+    """
+  end
+end

test/algora/bounties_test.exs

@@ -75,7 +75,7 @@ defmodule Algora.BountiesTest do
                  %{
                    owner: owner,
                    amount: ~M[4000]usd,
-                   bounty_id: bounty.id,
+                   bounty: bounty,
                    claims: claims
                  },
                  installation_id: installation.id
@@ -193,7 +193,7 @@ defmodule Algora.BountiesTest do
                Bounties.create_invoice(
                  %{owner: owner, amount: amount, idempotency_key: "bounty-#{bounty.id}"},
                  ticket_ref: ticket_ref,
-                 bounty_id: bounty.id,
+                 bounty: bounty,
                  claims: [claim]
                )