Sync team consistently on site creation (plausible#4736)

yonasBSD · Oct 24, 2024 · 0404522 · 0404522
1 parent 2ec9e32
commit 0404522
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/lib/plausible/sites.ex b/lib/plausible/sites.ex
@@ -206,13 +206,9 @@ defmodule Plausible.Sites do
         Site.Membership.new(site, user)
       end)
       |> maybe_start_trial(user)
-      |> Ecto.Multi.run(:sync_team, fn
-        _repo, %{user: user} ->
-          Plausible.Teams.sync_team(user)
-          {:ok, nil}
-
-        _repo, _context ->
-          {:ok, nil}
+      |> Ecto.Multi.run(:sync_team, fn _repo, %{user: user} ->
+        Plausible.Teams.sync_team(user)
+        {:ok, nil}
       end)
       |> Repo.transaction()
     end
@@ -226,7 +222,7 @@ defmodule Plausible.Sites do
         end)
 
       _ ->
-        multi
+        Ecto.Multi.put(multi, :user, user)
     end
   end