feat: allow attaching same payment method or customer record to multiple orgs (#67)

Author: zcesur

lib/algora/payments/schemas/customer.ex

@@ -28,7 +28,6 @@ defmodule Algora.Payments.Customer do
     |> cast(attrs, [:user_id, :provider, :provider_id, :provider_meta, :name])
     |> generate_id()
     |> validate_required([:user_id, :provider, :provider_id, :provider_meta, :name])
-    |> unique_constraint([:provider, :provider_id])
     |> unique_constraint(:user_id)
     |> foreign_key_constraint(:user_id)
   end

lib/algora/payments/schemas/payment_method.ex

@@ -23,7 +23,6 @@ defmodule Algora.Payments.PaymentMethod do
     |> cast(attrs, [:provider, :provider_id, :provider_meta, :provider_customer_id, :is_default, :customer_id])
     |> generate_id()
     |> validate_required([:provider, :provider_id, :provider_meta, :provider_customer_id, :is_default, :customer_id])
-    |> unique_constraint([:provider, :provider_id])
     |> foreign_key_constraint(:customer_id)
   end
 end

priv/repo/migrations/20250228212307_drop_provider_indexes.exs

@@ -0,0 +1,13 @@
+defmodule Algora.Repo.Migrations.DropProviderIndexes do
+  use Ecto.Migration
+
+  def up do
+    drop unique_index(:payment_methods, [:provider, :provider_id])
+    drop unique_index(:customers, [:provider, :provider_id])
+  end
+
+  def down do
+    create unique_index(:payment_methods, [:provider, :provider_id])
+    create unique_index(:customers, [:provider, :provider_id])
+  end
+end

scripts/database_migration.exs

@@ -73,6 +73,8 @@ defmodule DatabaseMigration do
     {"Reward", ["bounty_id"]}
   ]
 
+  @test_org_id "cljo6j981000el60f1k1cvtns"
+
   defp relevant_tables do
     @schema_mappings
     |> Enum.map(fn {k, _v} -> k end)
@@ -699,7 +701,7 @@ defmodule DatabaseMigration do
       raise "Owner not found: #{inspect(row)}"
     end
 
-    if owner["id"] not in ["clfqtao4h0001mo0gkp9az0bn", "cm251pvg40007ld031q5t2hj2", "cljo6j981000el60f1k1cvtns"] do
+    if owner["id"] != @test_org_id do
       %{
         "id" => row["id"],
         "provider" => "stripe",
@@ -726,7 +728,7 @@ defmodule DatabaseMigration do
       raise "StripeCustomer not found: #{inspect(row)}"
     end
 
-    if owner["id"] not in ["clfqtao4h0001mo0gkp9az0bn", "cm251pvg40007ld031q5t2hj2", "cljo6j981000el60f1k1cvtns"] do
+    if owner["id"] != @test_org_id do
       %{
         "id" => row["id"],
         "provider" => "stripe",