Graham McIntire
fd976b0cd5
- Add jump_credo_checks ~> 0.4 with all 20 checks enabled - Fix all standard Credo issues: 139 @spec (113 done, 26 remain), 4 refactoring, 3 alias usage, 9 System.cmd env, 5 unsafe_to_atom, 2 max line length, 9 assert_receive timeout - Fix 170+ jump_credo_checks warnings: - 117 TopLevelAliasImportRequire: move nested alias/import to module top - 32 UseObanProWorker: switch to Oban.Pro.Worker - 4 DoctestIExExamples: add doctests / create test file - ~20 WeakAssertion: strengthen type-check assertions - Various ConditionalAssertion, AssertReceiveTimeout fixes - Exclude vendor/ from Credo analysis - Remaining: 175 warnings (mostly opinionated WeakAssertion, AvoidSocketAssignsInTest), 26 @spec annotations
34 lines
1,008 B
Elixir
34 lines
1,008 B
Elixir
defmodule MicrowavepropWeb.UserRegistrationController do
|
|
use MicrowavepropWeb, :controller
|
|
|
|
alias Microwaveprop.Accounts
|
|
alias Microwaveprop.Accounts.User
|
|
|
|
@spec new(Plug.Conn.t(), map()) :: Plug.Conn.t()
|
|
def new(conn, _params) do
|
|
changeset = Accounts.change_user_registration(%User{})
|
|
render(conn, :new, changeset: changeset)
|
|
end
|
|
|
|
@spec create(Plug.Conn.t(), map()) :: Plug.Conn.t()
|
|
def create(conn, %{"user" => user_params}) do
|
|
case Accounts.register_user(user_params) do
|
|
{:ok, user} ->
|
|
{:ok, _email} =
|
|
Accounts.deliver_user_confirmation_instructions(
|
|
user,
|
|
&url(~p"/users/confirm/#{&1}")
|
|
)
|
|
|
|
conn
|
|
|> put_flash(
|
|
:info,
|
|
"Account created. Please check #{user.email} for a confirmation link to activate your account."
|
|
)
|
|
|> redirect(to: ~p"/users/log-in")
|
|
|
|
{:error, %Ecto.Changeset{} = changeset} ->
|
|
render(conn, :new, changeset: changeset)
|
|
end
|
|
end
|
|
end
|