Users can debrief early

Addresses #209
TechforgoodCAST · Nov 26, 2019 · b9a2e49 · b9a2e49
1 parent c30dc8b
commit b9a2e49
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/app/views/iterations/show.html.erb b/app/views/iterations/show.html.erb
@@ -42,7 +42,7 @@
           <div class="w-full sm:w-3/4 mb-3">
             Every two weeks
             <% if policy(@iteration).create? %>
-              <small>(Only visible to project contributors)</small>
+              <br><small>(Only visible to project contributors)</small>
             <% end %>
           </div>
         </div>
@@ -74,7 +74,12 @@
             Debrief
           </div>
           <div class="w-full sm:w-3/4 mb-3">
-            Due <%= with_default(@iteration.planned_debrief_date, formatter: :friendly_date) %>
+            Due <%= with_default(@iteration.planned_debrief_date, formatter: :friendly_date) %><br>
+            <% if @iteration.planned_debrief_date.future? %> 
+				<small class="missing-text">Finished learning? <%= link_to 'Debrief early', new_project_iteration_debrief_path(@project, @iteration), class: 'link' %></small>
+			<% else %>
+				<small class="text-danger">Overdue! <%= link_to 'Add a debrief', new_project_iteration_debrief_path(@project, @iteration), class: 'link' %></small>
+			<% end %>
           </div>
         <% else %>
           <div class="w-full sm:w-1/4 mb-3">

diff --git a/test/system/debriefs_test.rb b/test/system/debriefs_test.rb
@@ -21,6 +21,12 @@ class DebriefsTest < ApplicationSystemTestCase
     visit new_project_iteration_debrief_path(@project, @iteration)
     assert_text('New debrief')
   end
+
+  test 'contributors can debrief early' do
+    Membership.last.update(role: :contributor)
+    visit project_iteration_path(@project, @iteration)
+    assert_text('Debrief early')
+  end
 
   test 'contributors can view debriefs' do
     @debrief.save!