completes #84 : interview request status tweaks

app/models/candidate.rb

@@ -23,6 +23,11 @@ class Candidate < ApplicationRecord
     declined: 2
   }
 
+  def interview?
+    return 'yes' if approved?
+    'no' if declined?
+  end
+
   def build_reviews
     reviewers.each do |reviewer|
       votes.find_or_create_by(user_id: reviewer.id)

app/views/admin/candidate/index.html.erb

@@ -16,7 +16,7 @@
       <th>Progress</th>
       <th>Completed</th>
       <th>Reminded</th>
-      <th>Interview Request</th>
+      <th>Interview?</th>
     </tr>
 
     <% @candidates.each do |candidate| %>
@@ -32,7 +32,7 @@
         <td><%= candidate.status %></td>
         <td><%= candidate.completed ? link_to("Submitted", admin_result_path(candidate.test_hash)) : "" %></td>
         <td><%= candidate.reminded ? "Yes" : "" %></td>
-        <td><%= candidate.review_status unless candidate.pending? %></td>
+        <td><%= candidate.interview? %></td>
       </tr>
     <% end %>
   </table>

app/views/admin/result/_voting.html.erb

@@ -1,4 +1,4 @@
-<% # TODO: This needs to be extracted into a decorator, or something. It is only a quick hack solution. %>
+<% # TODO: This should be extracted into a decorator, or something. It is only a quick hack solution. %>
 
 <% if current_user.acts_as_reviewer? %>
   <div class="review_meta__votes" data-id="vote-count">
@@ -15,19 +15,22 @@
 
 <% if current_user.acts_as_manager? %>
   <div class="review_meta__vetos" data-id="veto-status">
-    <strong>Manager Vetos: </strong>
+    <strong>Interview: </strong>
-    <%= link_to admin_approve_vote_path(test_hash: @candidate.test_hash), remote: true do %>
-      <span data-id="interview-request">
-        <%= @candidate.approved? ? "Requested" : "Request Interview" %>
-      </span>
-    <% end %>
-    <%= link_to admin_decline_vote_path(test_hash: @candidate.test_hash), remote: true do %>
-      <span data-id="interview-decline">
-        <%= @candidate.declined? ? "Declined" : "Decline Interview" %>
-      </span>
-    <% end %>
-  </div>
 
+    <% if !@candidate.approved? %>
+      <%= link_to admin_approve_vote_path(test_hash: @candidate.test_hash), remote: true do %>
+        <span data-id="interview-request">Yes</span>
+      <% end %>
+    <% else %>Yes<% end %>
+
+
+    <% if !@candidate.declined? %>
+      <%= link_to admin_decline_vote_path(test_hash: @candidate.test_hash), remote: true do %>
+        <span data-id="interview-decline">No</span>
+      <% end %>
+    <% else %>No<% end %>
+
+  </div>
 <% else %>
 
   <strong>Candidate Interview Status: </strong><%= @candidate.review_status %>

app/views/admin/result/index.html.erb

@@ -8,7 +8,7 @@
       <th>Test ID</th>
       <th>Experience</th>
       <th>Recruiter</th>
-      <th>Interview Request</th>
+      <th>Interview?</th>
     </tr>
 
     <% @candidates.each do |candidate| %>
@@ -16,7 +16,7 @@
         <td><%= link_to candidate.test_hash, admin_result_path(candidate.test_hash) %></td>
         <td><%= candidate.experience %> years</td>
         <td><%= mail_to(candidate.recruiter.email) %></td>
-        <td><%= candidate.review_status unless candidate.pending? %></td>
+        <td><%= candidate.interview? %></td>
       </tr>
     <% end %>
   </table>