[129, 60, 130] Evaluation Form Model, Migration, CRUD, Responsive List View

app/controllers/evaluation_forms_controller.rb

@@ -1,5 +1,97 @@
 # frozen_string_literal: true
 
 class EvaluationFormsController < ApplicationController
-  def index; end
+  before_action :set_evaluation_form, only: %i[show edit update destroy]
+
+  # GET /evaluation_forms or /evaluation_forms.json
+  def index
+    @evaluation_forms = EvaluationForm.all
+  end
+
+  # GET /evaluation_forms/1 or /evaluation_forms/1.json
+  def show; end
+
+  # GET /evaluation_forms/new
+  def new
+    @evaluation_form = EvaluationForm.new
+  end
+
+  # GET /evaluation_forms/1/edit
+  def edit; end
+
+  # POST /evaluation_forms or /evaluation_forms.json
+  def create
+    @evaluation_form = EvaluationForm.new(evaluation_form_params)
+
+    respond_to do |format|
+      if @evaluation_form.save
+        format.html do
+          redirect_to evaluation_form_url(@evaluation_form), notice: "Evaluation form was successfully created."
+        end
+        format.json { render :show, status: :created, location: @evaluation_form }
+      else
+        format.html { render :new, status: :unprocessable_entity }
+        format.json { render json: @evaluation_form.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # NOTE: to reviewer: the following method works but it's impossible to use because there's a unique index on
+  # challenge_id and challenge_phase. So we can't clone evaluation forms without automatically advancing the challenge phase,
+  # or something along those lines. Also, we're leaving out the clone button which would trigger this for now,
+  # but I'm leaving it in case the button gets implemented later.
+
+  def create_from_existing
+    @existing_form = EvaluationForm.find(params[:evaluation_form])
+    @evaluation_form = EvaluationForm.new(@existing_form.attributes.except("id"))
+    respond_to do |format|
+      if @evaluation_form.save
+        format.html do
+          redirect_to evaluation_forms_url, notice: "Evaluation form was successfully cloned."
+        end
+        format.json { render :show, status: :created, location: @evaluation_form }
+      else
+        format.html { render :index, status: :unprocessable_entity }
+        format.json { render json: @evaluation_form.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # PATCH/PUT /evaluation_forms/1 or /evaluation_forms/1.json
+  def update
+    respond_to do |format|
+      if @evaluation_form.update(evaluation_form_params)
+        format.html do
+          redirect_to evaluation_form_url(@evaluation_form), notice: "Evaluation form was successfully updated."
+        end
+        format.json { render :show, status: :ok, location: @evaluation_form }
+      else
+        format.html { render :edit, status: :unprocessable_entity }
+        format.json { render json: @evaluation_form.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # DELETE /evaluation_forms/1 or /evaluation_forms/1.json
+  def destroy
+    @evaluation_form.destroy!
+
+    respond_to do |format|
+      format.html { redirect_to evaluation_forms_url, notice: "Evaluation form was successfully destroyed." }
+      format.json { head(:no_content) }
+    end
+  end
+
+  private
+
+  # Use callbacks to share common setup or constraints between actions.
+  def set_evaluation_form
+    @evaluation_form = EvaluationForm.find(params[:id])
+  end
+
+  # Only allow a list of trusted parameters through.
+  def evaluation_form_params
+    params.require(:evaluation_form).permit(:title, :instructions, :challenge_phase, :status, :comments_required,
+                                            :weighted_scoring, :publication_date, :closing_date, :challenge_id)
+  end
 end

app/helpers/evaluation_forms_helper.rb

@@ -1,4 +1,15 @@
 # frozen_string_literal: true
 
 module EvaluationFormsHelper
+  def status_colors
+    {
+      draft: "FireBrick",
+      ready: "DarkGoldenRod",
+      published: "green"
+    }
+  end
+
+  def challenge_with_phase(evaluation_form)
+    "#{evaluation_form.challenge.title} - Phase #{evaluation_form.challenge_phase}"
+  end  
 end

app/models/evaluation_form.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class EvaluationForm < ApplicationRecord
+  belongs_to :challenge
+
+  enum :status, { draft: 0, ready: 1, published: 2 }
+
+  validates :title, presence: true
+  validates :instructions, presence: true
+  validates :challenge_phase, presence: true
+end

app/views/evaluation_forms/_evaluation_form.html.erb

@@ -0,0 +1,47 @@
+<div id="<%= dom_id evaluation_form %>">
+  <p>
+    <strong>Title:</strong>
+    <%= evaluation_form.title %>
+  </p>
+
+  <p>
+    <strong>Instructions:</strong>
+    <%= evaluation_form.instructions %>
+  </p>
+
+  <p>
+    <strong>Challenge phase:</strong>
+    <%= evaluation_form.challenge_phase %>
+  </p>
+
+  <p>
+    <strong>Status:</strong>
+    <%= evaluation_form.status %>
+  </p>
+
+  <p>
+    <strong>Comments required:</strong>
+    <%= evaluation_form.comments_required %>
+  </p>
+
+  <p>
+    <strong>Weighted scoring:</strong>
+    <%= evaluation_form.weighted_scoring %>
+  </p>
+
+  <p>
+    <strong>Publication date:</strong>
+    <%= evaluation_form.publication_date %>
+  </p>
+
+  <p>
+    <strong>Closing date:</strong>
+    <%= evaluation_form.closing_date %>
+  </p>
+
+  <p>
+    <strong>Challenge:</strong>
+    <%= evaluation_form.challenge_id %>
+  </p>
+
+</div>

app/views/evaluation_forms/_evaluation_form.json.jbuilder

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+json.extract!(evaluation_form, :id, :title, :instructions, :challenge_phase, :status, :comments_required,
+              :weighted_scoring, :publication_date, :closing_date, :challenge_id, :created_at, :updated_at)
+json.url(evaluation_form_url(evaluation_form, format: :json))

app/views/evaluation_forms/_form.html.erb

@@ -0,0 +1,62 @@
+<%= form_with(model: evaluation_form) do |form| %>
+  <% if evaluation_form.errors.any? %>
+    <div style="color: red">
+      <h2><%= pluralize(evaluation_form.errors.count, "error") %> prohibited this evaluation_form from being saved:</h2>
+
+      <ul>
+        <% evaluation_form.errors.each do |error| %>
+          <li><%= error.full_message %></li>
+        <% end %>
+      </ul>
+    </div>
+  <% end %>
+
+  <div>
+    <%= form.label :title, style: "display: block" %>
+    <%= form.text_field :title %>
+  </div>
+
+  <div>
+    <%= form.label :instructions, style: "display: block" %>
+    <%= form.text_field :instructions %>
+  </div>
+
+  <div>
+    <%= form.label :challenge_phase, style: "display: block" %>
+    <%= form.number_field :challenge_phase %>
+  </div>
+
+  <div>
+    <%= form.label :status, style: "display: block" %>
+    <%=  form.collection_select :status, EvaluationForm.statuses.map{ |a| [a.first,a.first] },  :first, :second %>
+  </div>
+
+  <div>
+    <%= form.label :comments_required, style: "display: block" %>
+    <%= form.check_box :comments_required %>
+  </div>
+
+  <div>
+    <%= form.label :weighted_scoring, style: "display: block" %>
+    <%= form.check_box :weighted_scoring %>
+  </div>
+
+  <div>
+    <%= form.label :publication_date, style: "display: block" %>
+    <%= form.date_field :publication_date %>
+  </div>
+
+  <div>
+    <%= form.label :closing_date, style: "display: block" %>
+    <%= form.date_field :closing_date %>
+  </div>
+
+  <div>
+    <%= form.label :challenge_id, style: "display: block" %>
+    <%= form.text_field :challenge_id %>
+  </div>
+
+  <div>
+    <%= form.submit %>
+  </div>
+<% end %>

app/views/evaluation_forms/_table.html.erb

@@ -0,0 +1,49 @@
+<table class="usa-table usa-table--stacked-header usa-table--borderless width-full">
+    <thead>
+      <tr>
+        <th scope="col">Form Title</th>
+        <th scope="col">Assigned to Challenge:</th>
+        <th scope="col">Status:</th>
+        <th scope="col"></th>
+      </tr>
+    </thead>
+    <tbody>
+      <% @evaluation_forms.each do |evaluation_form| %>
+        <tr>
+          <th data-label="Form Title" scope="row">
+            <%= link_to evaluation_form do %>
+                <%= evaluation_form.title %>
+            <% end %>    
+            </th>
+            <td data-label="Assigned to Challenge:">
+                <%= challenge_with_phase(evaluation_form) %>
+            </td>
+          <td data-label="Status:">
+            <span style="<%="color: #{status_colors[evaluation_form.status.to_sym]}"%>">
+                <%= evaluation_form.status.capitalize %>
+            </span>
+          </td>
+          <td>
+              <div class="display-flex flex-wrap grid-row grid-gap-1">
+                <%= button_to edit_evaluation_form_path(evaluation_form), method: :get, class: "usa-button width-full", form: {class: "grid-col-6"} do %>
+                  <%= image_tag(
+                    "images/usa-icons/content_copy.svg",
+                    class: "usa-icon icon-white",
+                    alt: "evaluation forms"
+                  )%>
+                  Edit
+                <% end %>
+                <%= button_to evaluation_form, method: :delete, class: "usa-button usa-button--secondary width-full padding-x-1", form: {class: "grid-col-6"} do %>
+                  <%= image_tag(
+                    "images/usa-icons/content_copy.svg",
+                    class: "usa-icon icon-white",
+                    alt: "evaluation forms"
+                  )%>
+                  Delete
+                <% end %>
+              </div>  
+          </td>  
+        </tr>
+      <% end %>  
+    </tbody>
+  </table>

app/views/evaluation_forms/edit.html.erb

@@ -0,0 +1,12 @@
+<% content_for :title, "Editing evaluation form" %>
+
+<h1>Editing evaluation form</h1>
+
+<%= render "form", evaluation_form: @evaluation_form %>
+
+<br>
+
+<div>
+  <%= link_to "Show this evaluation form", @evaluation_form %> |
+  <%= link_to "Back to evaluation forms", evaluation_forms_path %>
+</div>

app/views/evaluation_forms/index.html.erb

@@ -1,7 +1,27 @@
-<div class="usa-card__container col-md-6">
-  <div class="usa-card__body">
-    <h1>Evaluation Forms</h1>
+<div class="usa-card__body">
+  <h1>Evaluation Forms</h1>
+  <div class="display-flex flex-wrap tablet:flex-justify width-full">
+    <span class="text-base margin-bottom-1">Manage existing evaluation forms and create new evaluation forms.</span>
+
+        <%= link_to new_evaluation_form_path, class: "width-full tablet:width-auto" do %>
+          <button class="usa-button font-body-2xs text-no-wrap" style="background-color: #4d8055">
+            <%= image_tag(
+              "images/usa-icons/content_copy.svg",
+              class: "usa-icon icon-white",
+              alt: "evaluation forms"
+            )%>
+            Create New Evaluation Form
+          </button> 
+    <% end %>
+  </div>  
+
+  <% if @evaluation_forms.empty? %>
+    <div class="text-base align-center text-center">
+      <p>You currently do not have any evaluation forms.</p>
+      <p>Please create an evaluation form for your challenge before it is published.</p>
+    </div>  
+  <% else %>
+    <%= render partial: "table", locals: { evaluation_forms: @evaluation_forms } %>
+  <% end %>  
 
-    <p>Placeholder text for Evaluation Forms.</p>
-  </div>
 </div>

app/views/evaluation_forms/index.json.jbuilder

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+json.array!(@evaluation_forms, partial: "evaluation_forms/evaluation_form", as: :evaluation_form)

app/views/evaluation_forms/new.html.erb

@@ -0,0 +1,11 @@
+<% content_for :title, "New evaluation form" %>
+
+<h1>New evaluation form</h1>
+
+<%= render "form", evaluation_form: @evaluation_form %>
+
+<br>
+
+<div>
+  <%= link_to "Back to evaluation forms", evaluation_forms_path %>
+</div>

app/views/evaluation_forms/show.html.erb

@@ -0,0 +1,10 @@
+<p style="color: green"><%= notice %></p>
+
+<%= render @evaluation_form %>
+
+<div>
+  <%= link_to "Edit this evaluation form", edit_evaluation_form_path(@evaluation_form) %> |
+  <%= link_to "Back to evaluation forms", evaluation_forms_path %>
+
+  <%= button_to "Destroy this evaluation form", @evaluation_form, method: :delete %>
+</div>

app/views/evaluation_forms/show.json.jbuilder

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+json.partial!("evaluation_forms/evaluation_form", evaluation_form: @evaluation_form)

config/routes.rb

@@ -11,7 +11,8 @@
   get '/dashboard', to: "dashboard#index"
 
   resources :evaluations, only: [:index]
-  resources :evaluation_forms, only: [:index]
+  resources :evaluation_forms
+  post '/evaluation_forms/clone', to: 'evaluation_forms#create_from_existing'
   resources :manage_submissions, only: [:index]
 
   # Reveal health status on /up that returns 200 if the app boots with no exceptions, otherwise 500.