address comments

Author: SophieDeBenedetto

lib/pointing_party_web/controllers/session_controller.ex

@@ -10,8 +10,8 @@ defmodule PointingPartyWeb.SessionController do
 
   def create(conn, params) do
     case Auth.login(params["account"]) do
-      {:ok, account} ->
-        put_session(conn, :username, account.username)
+      {:ok, %{username: username}} ->
+        put_session(conn, :username, username)
         |> redirect(to: "/cards") |> halt()
       {:error, changeset} ->
         render(conn, "new.html", changeset: changeset)

lib/pointing_party_web/plugs/auth.ex

@@ -6,7 +6,7 @@ defmodule PointingPartyWeb.Plugs.Auth do
 
   def call(conn, _default) do
     case authenticate(conn) do
-      nil -> redirect(conn, to: "/login") |> halt()
+      nil -> conn |> redirect(to: "/login") |> halt()
       username -> assign(conn, :username, username)
     end
   end

lib/pointing_party_web/views/layout_helpers.ex

@@ -5,6 +5,6 @@ defmodule PointingPartyWeb.LayoutHelpers do
   Generates tag for inlined form input errors.
   """
   def signed_in?(conn) do
-    conn.assigns[:username]
+    not is_nil(conn.assigns[:username])
   end
 end