Public registration changed from performers to visitors.

lib/runosaari_web/controllers/visitor_controller.ex

@@ -18,8 +18,7 @@ defmodule RunosaariWeb.VisitorController do
     case Registration.create_visitor(visitor_params) do
       {:ok, visitor} ->
         conn
-        |> put_flash(:info, "Visitor created successfully.")
+        |> redirect(to: Routes.visitor_path(conn, :confirmation))
-        |> redirect(to: Routes.visitor_path(conn, :show, visitor))
 
       {:error, %Ecto.Changeset{} = changeset} ->
         render(conn, "new.html", changeset: changeset)
@@ -59,4 +58,8 @@ defmodule RunosaariWeb.VisitorController do
     |> put_flash(:info, "Visitor deleted successfully.")
     |> redirect(to: Routes.visitor_path(conn, :index))
   end
+
+  def confirmation(conn, _params) do
+    render(conn, "confirmation.html")
+  end
 end

lib/runosaari_web/router.ex

@@ -21,6 +21,8 @@ defmodule RunosaariWeb.Router do
     get "/covid19", PageController, :covid19
     resources "/performers", PerformerController, only: [:index, :show]
     resources "/performances", PerformanceController, only: [:index, :show]
+    resources "/visitors", VisitorController, only: [:new, :create]
+    get "/confirmation", VisitorController, :confirmation
   end
 
   scope "/admin", RunosaariWeb, as: :admin do
@@ -30,6 +32,7 @@ defmodule RunosaariWeb.Router do
     get "/performances", PerformanceController, :admin
     resources "/performers", PerformerController, except: [:index, :show]
     resources "/performances", PerformanceController, except: [:index, :show]
+    resources "/visitors", VisitorController, except: [:new, :create]
   end
 
   # Other scopes may use custom stacks.

lib/runosaari_web/templates/layout/app.html.eex

@@ -20,7 +20,7 @@
         <%= link "Esiintyjät", to: Routes.performer_path(@conn, :index) %>
         <%= link "Info", to: Routes.page_path(@conn, :info) %>
         <%= link "Covid-19", to: Routes.page_path(@conn, :covid19) %>
-        <%= link "Ilmoittautuminen", to: Routes.admin_performer_path(@conn, :new) %>
+        <%= link "Ilmoittautuminen", to: Routes.visitor_path(@conn, :new) %>
       </nav>
       <h1 class="mobile-main-title">
         <a href="/#logo-container">Runosaari 2021</a>
@@ -31,7 +31,7 @@
         <%= link "Esiintyjät", to: "#{Routes.performer_path(@conn, :index)}#performers-start" %>
         <%= link "Info", to: "#{Routes.page_path(@conn, :info)}#contact-start" %>
         <%= link "Covid-19", to: "#{Routes.page_path(@conn, :covid19)}#covid-start" %>
-        <%= link "Ilmoittautuminen", to: "#{Routes.admin_performer_path(@conn, :new)}#registration-start" %>
+        <%= link "Ilmoittautuminen", to: "#{Routes.visitor_path(@conn, :new)}#registration-start" %>
       </nav>
     </header>
     <main role="main">

lib/runosaari_web/templates/performer/new.html.eex

@@ -1,5 +1,5 @@
 <section class="text">
-    <h1 id="registration-start">Ilmoittaudu</h1>
+    <h1>Ilmoittaudu</h1>
 
     <%= render "form.html", Map.put(assigns, :action, Routes.admin_performer_path(@conn, :create)) %>
 </section>

lib/runosaari_web/templates/visitor/confirmation.html.eex

@@ -0,0 +1,5 @@
+<section class="text">
+    <h2>Ilmoittautumisesi on rekisteröity. </h2>
+    <h1>Tervetuloa mukaan!</h1>
+    <h3>Tapahtuman järjestäjät ovat teihin yhteydessä sähköpostilla tarpeen mukaan.</h3>
+</section>

lib/runosaari_web/templates/visitor/form.html.eex

@@ -1,47 +1,55 @@
-<%= form_for @changeset, @action, fn f -> %>
+<%= form_for @changeset, @action, fn form -> %>
   <%= if @changeset.action do %>
     <div class="alert alert-danger">
-      <p>Oops, something went wrong! Please check the errors below.</p>
+      <p>Jokin kentistä on tyhjä.</p>
     </div>
   <% end %>
 
-  <%= label f, :fname %>
+  <%= label form, :fname, "Etunimi" %>
-  <%= text_input f, :fname %>
+  <%= text_input form, :fname %>
-  <%= error_tag f, :fname %>
+  <%= error_tag form, :fname %>
 
-  <%= label f, :lname %>
+  <%= label form, :lname, "Sukunimi" %>
-  <%= text_input f, :lname %>
+  <%= text_input form, :lname %>
-  <%= error_tag f, :lname %>
+  <%= error_tag form, :lname %>
 
-  <%= label f, :email %>
+  <%= label form, :email, "Sähköposti" %>
-  <%= text_input f, :email %>
+  <%= email_input form, :email %>
-  <%= error_tag f, :email %>
+  <%= error_tag form, :email %>
 
-  <%= label f, :tel %>
+  <%= label form, :tel, "Puhelinumero" %>
-  <%= text_input f, :tel %>
+  <%= text_input form, :tel %>
-  <%= error_tag f, :tel %>
+  <%= error_tag form, :tel %>
 
-  <%= label f, :date1 %>
+  <h2>Ilmoita alustava kiinnostuksesi tässä</h2>
-  <%= checkbox f, :date1 %>
-  <%= error_tag f, :date1 %>
 
-  <%= label f, :date2 %>
+  <div class="checkbox-container">
-  <%= checkbox f, :date2 %>
+    <%= checkbox form, :date1 %>
-  <%= error_tag f, :date2 %>
+    <%= label form, :date1, "Torstai 22.7" %>
-
+    <%= error_tag form, :date1 %>
-  <%= label f, :date3 %>
+  </div>
-  <%= checkbox f, :date3 %>
+  <div class="checkbox-container">
-  <%= error_tag f, :date3 %>
+    <%= checkbox form, :date2 %>
-
+    <%= label form, :date2, "Perjantai 23.7." %>
-  <%= label f, :bus %>
+    <%= error_tag form, :date2 %>
-  <%= checkbox f, :bus %>
+  </div>
-  <%= error_tag f, :bus %>
+  <div class="checkbox-container">
-
+    <%= checkbox form, :date3 %>
-  <%= label f, :accom %>
+    <%= label form, :date3, "Lauantai 24.7." %>
-  <%= checkbox f, :accom %>
+    <%= error_tag form, :date3 %>
-  <%= error_tag f, :accom %>
+  </div>
+  <div class="checkbox-container">
+    <%= checkbox form, :bus %>
+    <%= label form, :bus, "Olen kiinnostunut bussikuljetuksesta" %>
+    <%= error_tag form, :bus %>
+  </div>
+  <div class="checkbox-container">
+    <%= checkbox form, :accom %>
+    <%= label form, :accom, "Olen kiinnostunut majoituksesta" %>
+    <%= error_tag form, :accom %>
+  </div>
 
   <div>
-    <%= submit "Save" %>
+    <%= submit "Lähetä" %>
   </div>
 <% end %>

lib/runosaari_web/templates/visitor/new.html.eex

@@ -1,5 +1,5 @@
-<h1>New Visitor</h1>
+<section class="text">
+    <h1 id="registration-start">Ilmoittaudu vierailijaksi</h1>
 
-<%= render "form.html", Map.put(assigns, :action, Routes.visitor_path(@conn, :create)) %>
+    <%= render "form.html", Map.put(assigns, :action, Routes.visitor_path(@conn, :create)) %>
-
+</section>
-<span><%= link "Back", to: Routes.visitor_path(@conn, :index) %></span>

test/runosaari_web/controllers/visitor_controller_test.exs

@@ -3,9 +3,39 @@ defmodule RunosaariWeb.VisitorControllerTest do
 
   alias Runosaari.Registration
 
-  @create_attrs %{accom: true, bus: true, date1: true, date2: true, date3: true, email: "some email", fname: "some fname", lname: "some lname", tel: "some tel"}
+  @create_attrs %{
-  @update_attrs %{accom: false, bus: false, date1: false, date2: false, date3: false, email: "some updated email", fname: "some updated fname", lname: "some updated lname", tel: "some updated tel"}
+    accom: true,
-  @invalid_attrs %{accom: nil, bus: nil, date1: nil, date2: nil, date3: nil, email: nil, fname: nil, lname: nil, tel: nil}
+    bus: true,
+    date1: true,
+    date2: true,
+    date3: true,
+    email: "some email",
+    fname: "some fname",
+    lname: "some lname",
+    tel: "some tel"
+  }
+  @update_attrs %{
+    accom: false,
+    bus: false,
+    date1: false,
+    date2: false,
+    date3: false,
+    email: "some updated email",
+    fname: "some updated fname",
+    lname: "some updated lname",
+    tel: "some updated tel"
+  }
+  @invalid_attrs %{
+    accom: nil,
+    bus: nil,
+    date1: nil,
+    date2: nil,
+    date3: nil,
+    email: nil,
+    fname: nil,
+    lname: nil,
+    tel: nil
+  }
 
   def fixture(:visitor) do
     {:ok, visitor} = Registration.create_visitor(@create_attrs)
@@ -57,6 +87,7 @@ defmodule RunosaariWeb.VisitorControllerTest do
 
     test "redirects when data is valid", %{conn: conn, visitor: visitor} do
       conn = put(conn, Routes.visitor_path(conn, :update, visitor), visitor: @update_attrs)
+
       assert redirected_to(conn) == Routes.visitor_path(conn, :show, visitor)
 
       conn = get(conn, Routes.visitor_path(conn, :show, visitor))
@@ -65,6 +96,7 @@ defmodule RunosaariWeb.VisitorControllerTest do
 
     test "renders errors when data is invalid", %{conn: conn, visitor: visitor} do
       conn = put(conn, Routes.visitor_path(conn, :update, visitor), visitor: @invalid_attrs)
+
       assert html_response(conn, 200) =~ "Edit Visitor"
     end
   end
@@ -75,6 +107,7 @@ defmodule RunosaariWeb.VisitorControllerTest do
     test "deletes chosen visitor", %{conn: conn, visitor: visitor} do
       conn = delete(conn, Routes.visitor_path(conn, :delete, visitor))
       assert redirected_to(conn) == Routes.visitor_path(conn, :index)
+
       assert_error_sent 404, fn ->
         get(conn, Routes.visitor_path(conn, :show, visitor))
       end