Don't ignore email error on user creation

beyond-all-reason · Jun 3, 2024 · d3ba30b · d3ba30b
1 parent b2dbd23
commit d3ba30b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/teiserver/common/email_helper.ex b/lib/teiserver/common/email_helper.ex
@@ -69,6 +69,6 @@ defmodule Teiserver.EmailHelper do
     |> Email.put_header("Message-Id", message_id)
     |> Email.html_body(html_body)
     |> Email.text_body(text_body)
-    |> Teiserver.Mailer.deliver_now()
+    |> Teiserver.Mailer.deliver_now(response: true)
   end
 end
diff --git a/lib/teiserver/data/cache_user.ex b/lib/teiserver/data/cache_user.ex
@@ -296,9 +296,8 @@ defmodule Teiserver.CacheUser do
               :no_verify ->
                 verify_user(get_user_by_id(user.id))
 
-              {:ok, _} ->
+              {:ok, _, _} ->
                 :ok
-                # Logger.error("Email sent, response of #{Kernel.inspect response}")
             end
         end
 
@@ -339,7 +338,7 @@ defmodule Teiserver.CacheUser do
               verify_user(get_user_by_id(user.id))
               :ok
 
-            {:ok, _} ->
+            {:ok, _, _} ->
               :ok
           end
         end