Merge pull request #5 from elixirschool/scallan/remove-db

Remove Repo from the project

Author: doomspork

config/cards.exs

@@ -0,0 +1,38 @@
+use Mix.Config
+
+config :pointing_party,
+  cards: [
+    %{
+      title: "Support loading data from a database",
+      description: """
+      We need to update the application to read users and cards from the database. This work includes adding Repo to the
+      project, creating migrations, and a seed.exs file.
+      """
+    },
+    %{
+      title: "Save pointing results to database",
+      description: """
+      Update the application to save an individual's vote and the final card points to the database.
+      """
+    },
+
+    %{
+      title: "Add Guardian dependency",
+      description: """
+      In preparation for adding authentication to the application, we need to add the Guardian dependency.
+      """
+    },
+    %{
+      title: "Create Auth module and config",
+      description: """
+      Create a module that uses the Guardian package and implements the approach set of callbacks. Additionally,
+      add the Guardian configuration.
+      """
+    },
+    %{
+      title: "Update authenication flow to use new auth",
+      description: """
+      Update our existing authentication flow to use the newly created Auth module.
+      """
+    },
+  ]

config/config.exs

@@ -1,10 +1,3 @@
-# This file is responsible for configuring your application
-# and its dependencies with the aid of the Mix.Config module.
-#
-# This configuration file is loaded before any dependency and
-# is restricted to this project.
-
-# General application configuration
 use Mix.Config
 
 config :pointing_party,
@@ -25,6 +18,8 @@ config :logger, :console,
 # Use Jason for JSON parsing in Phoenix
 config :phoenix, :json_library, Jason
 
+import_config "cards.exs"
+
 # Import environment specific config. This must remain at the bottom
 # of this file so it overrides the configuration defined above.
 import_config "#{Mix.env()}.exs"

config/dev.exs

@@ -1,17 +1,5 @@
 use Mix.Config
 
-# Configure your database
-config :pointing_party, PointingParty.Repo,
-  username: "postgres",
-  password: "postgres",
-  database: "pointing_party_dev",
-  hostname: "localhost",
-  show_sensitive_data_on_connection_error: true,
-  pool_size: 10
-
-# For development, we disable any cache and enable
-# debugging and code reloading.
-#
 # The watchers configuration can be used to run external
 # watchers to your application. For example, we use it
 # with webpack to recompile .js and .css sources.

config/test.exs

@@ -1,13 +1,5 @@
 use Mix.Config
 
-# Configure your database
-config :pointing_party, PointingParty.Repo,
-  username: "postgres",
-  password: "postgres",
-  database: "pointing_party_test",
-  hostname: "localhost",
-  pool: Ecto.Adapters.SQL.Sandbox
-
 # We don't run a server during test. If one is required,
 # you can enable the server option below.
 config :pointing_party, PointingPartyWeb.Endpoint,

lib/pointing_party/account.ex

@@ -1,6 +1,8 @@
 defmodule PointingParty.Account do
   use Ecto.Schema
+
   import Ecto.Changeset
+
   alias PointingParty.Account
 
   schema "accounts" do

lib/pointing_party/account/auth.ex

@@ -1,5 +1,6 @@
 defmodule PointingParty.Account.Auth do
   alias PointingParty.Account
+
   def login(params) do
     Account.create(params)
   end

lib/pointing_party/application.ex

@@ -7,7 +7,6 @@ defmodule PointingParty.Application do
 
   def start(_type, _args) do
     children = [
-      PointingParty.Repo,
       PointingPartyWeb.Endpoint,
       PointingPartyWeb.Presence
     ]

lib/pointing_party/card.ex

@@ -1,36 +1,33 @@
 defmodule PointingParty.Card do
   use Ecto.Schema
+
   import Ecto.Changeset
-  import Ecto.Query
-  alias PointingParty.Repo
-  alias PointingParty.Card
 
+  @pointing_scale [0, 1, 3, 5]
 
   schema "cards" do
     field :description, :string
-    field :title, :string
     field :points, :integer
+    field :title, :string
 
     timestamps()
   end
 
-  def points_range, do: [0, 1, 3, 5]
+  def points_range, do: @pointing_scale
 
   @doc false
   def changeset(card, attrs) do
     card
     |> cast(attrs, [:title, :description, :points])
     |> validate_required([:title, :description])
-    |> validate_inclusion(:points, Card.points_range())
+    |> validate_inclusion(:points, @pointing_scale)
   end
 
-  def get!(id) do
-    Repo.get!(Card, id)
-  end
+  def first, do: List.first(cards())
 
-  def first do
-    Card
-    |> first
-    |> Repo.one()
+  defp cards do
+    :pointing_party
+    |> Application.get_env(:cards)
+    |> Enum.map(&struct(__MODULE__, &1))
   end
 end

lib/pointing_party/repo.ex

@@ -17,9 +17,9 @@
             <%= if signed_in?(@conn) do %>
               <li>Hi, <%= @conn.assigns.username %></li>
               <li><%= link "Log Out", to: Routes.session_path(@conn, :delete), method: :delete, class: "nav-link" %></li>
-            <%= else %>
+            <% else %>
               <li><%= link "Log In", to: Routes.session_path(@conn, :new), class: "nav-link" %></li>
-            <%= end %>
+            <% end %>
           </ul>
         </nav>
         <a href="/" class="phx-logo">