Browse Source

Removing avatar

master
Pursuit 2 years ago
parent
commit
2f74e5a056
  1. 8
      lib/iqrcode/accounts/user.ex
  2. 15
      lib/iqrcode/accounts/user_from_auth.ex
  3. 2
      lib/iqrcode_web/templates/page/index.html.eex
  4. 10
      test/iqrcode/accounts_test.exs

8
lib/iqrcode/accounts/user.ex

@ -5,7 +5,7 @@ defmodule Iqrcode.Accounts.User do
alias Iqrcode.Content.Qrcode
schema "users" do
field :avatar, :string
#field :avatar, :string
field :email, :string
field :name, :string
has_many :qrcodes, Qrcode
@ -16,7 +16,9 @@ defmodule Iqrcode.Accounts.User do
@doc false
def changeset(user, attrs) do
user
|> cast(attrs, [:name, :avatar, :email])
|> validate_required([:name, :avatar, :email])
#|> cast(attrs, [:name, :avatar, :email])
#|> validate_required([:name, :avatar, :email])
|> cast(attrs, [:name, :email])
|> validate_required([:name, :email])
end
end

15
lib/iqrcode/accounts/user_from_auth.ex

@ -12,17 +12,18 @@ defmodule UserFromAuth do
end
# github does it this way
defp avatar_from_auth(%{info: %{urls: %{avatar_url: image}}}), do: image
#defp avatar_from_auth(%{info: %{urls: %{avatar_url: image}}}), do: image
# facebook does it this way
defp avatar_from_auth(%{info: %{image: image}}), do: image
#defp avatar_from_auth(%{info: %{image: image}}), do: image
# default case if nothing matches
defp avatar_from_auth(auth) do
Logger.warn(auth.provider <> " needs to find an avatar URL!")
Logger.debug(Jason.encode!(auth))
nil
end
#defp avatar_from_auth(auth) do
# Logger.warn(auth.provider <> " needs to find an avatar URL!")
# Logger.debug(Jason.encode!(auth))
# nil
#end
defp basic_info(auth) do
# Strips complex auth struct to fields in `user.ex`

2
lib/iqrcode_web/templates/page/index.html.eex

@ -25,7 +25,7 @@
<article class="column">
<%= if @current_user do %>
<h2>Welcome, <%= @current_user.name %>!</h2>
<img src="<%= @current_user.avatar %>" class="img-circle"/>
<!--<img src="<%= @current_user.avatar %>" class="img-circle"/>-->
<br>
<%= button "Logout", to: "/auth/logout", method: :delete, class: "btn btn-danger" %>
<% else %>

10
test/iqrcode/accounts_test.exs

@ -6,9 +6,11 @@ defmodule Iqrcode.AccountsTest do
describe "users" do
alias Iqrcode.Accounts.User
@valid_attrs %{avatar: "some avatar", email: "some email", name: "some name"}
@update_attrs %{avatar: "some updated avatar", email: "some updated email", name: "some updated name"}
#@valid_attrs %{avatar: "some avatar", email: "some email", name: "some name"}
#@update_attrs %{avatar: "some updated avatar", email: "some updated email", name: "some updated name"}
#@invalid_attrs %{avatar: nil, email: nil, name: nil}
@valid_attrs %{email: "some email", name: "some name"}
@update_attrs %{email: "some updated email", name: "some updated name"}
@invalid_attrs %{email: nil, name: nil}
def user_fixture(attrs \\ %{}) do
@ -32,7 +34,7 @@ defmodule Iqrcode.AccountsTest do
test "create_user/1 with valid data creates a user" do
assert {:ok, %User{} = user} = Accounts.create_user(@valid_attrs)
assert user.avatar == "some avatar"
#assert user.avatar == "some avatar"
assert user.email == "some email"
assert user.name == "some name"
end
@ -44,7 +46,7 @@ defmodule Iqrcode.AccountsTest do
test "update_user/2 with valid data updates the user" do
user = user_fixture()
assert {:ok, %User{} = user} = Accounts.update_user(user, @update_attrs)
assert user.avatar == "some updated avatar"
#assert user.avatar == "some updated avatar"
assert user.email == "some updated email"
assert user.name == "some updated name"
end

Loading…
Cancel
Save