From 598c1204e35eb1019d5964a71dd19e00380c0f90 Mon Sep 17 00:00:00 2001
From: Joshua Coles <josh@coles.to>
Date: Fri, 29 Sep 2023 21:32:28 +0100
Subject: [PATCH] Move the display to attendances

---
 app/jobs/scrape_toggl_job.rb                |  2 +
 app/views/attendance_tracker/index.html.erb | 63 ++++++++++-----------
 2 files changed, 32 insertions(+), 33 deletions(-)

diff --git a/app/jobs/scrape_toggl_job.rb b/app/jobs/scrape_toggl_job.rb
index a09c88b..159334c 100644
--- a/app/jobs/scrape_toggl_job.rb
+++ b/app/jobs/scrape_toggl_job.rb
@@ -25,6 +25,8 @@ class ScrapeTogglJob < ApplicationJob
     )
 
     lectures.each do |lecture|
+      return if lecture.attendance.present?
+
       entries_data.each do |entry|
         concurrent_time_entry = entry['time_entries'].find do |inner_entry|
           (Time.new(inner_entry['start']) - lecture.start_time) < 10.minutes
diff --git a/app/views/attendance_tracker/index.html.erb b/app/views/attendance_tracker/index.html.erb
index 4f28d14..3122718 100644
--- a/app/views/attendance_tracker/index.html.erb
+++ b/app/views/attendance_tracker/index.html.erb
@@ -17,8 +17,8 @@
                 <th scope="col" class="px-3 py-3.5 text-left text-sm font-semibold text-gray-900">Action</th>
               </tr>
               </thead>
-              <tbody class="divide-y divide-gray-200 bg-white">
 
+              <tbody class="divide-y divide-gray-200 bg-white">
               <% course.lectures.order(:start_time).group_by { |lecture| lecture.week_number }.each do |(week_number, lectures)| %>
                 <tr class="border-t border-gray-200">
                   <th colspan="4" scope="colgroup" class="bg-gray-50 py-2 pl-4 pr-3 text-left text-sm font-semibold text-gray-900 sm:pl-6">Week <%= week_number %></th>
@@ -32,39 +32,36 @@
                     <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
                       <%= lecture.start_time.to_fs(:dmy) %>
                     </td>
-                    <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
-                      <% if lecture.status == 'future' %>
-                        Future
-                      <% elsif lecture.status == 'happening_now' %>
-                        Happening Now
-                      <% elsif lecture.status == 'attended_in_person' %>
-                        <i class="fa fa-chalkboard-user fa-green-700"></i>
-                      <% elsif lecture.status == 'watched_recording' %>
-                        <i class="fa fa-video fa-green-700"></i>
-                      <% elsif lecture.status == 'missed' %>
-                        <i class="fa fa-times text-red-700"></i>
-                      <% else %>
-                        <% lecture.status %>
-                      <% end %>
-                    </td>
 
-                    <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
-                      <% if lecture.status == 'future' %>
-                      <% elsif lecture.status == 'happening_now' && lecture.live_video_url.present? %>
-                        <% link_to "Open live", lecture.live_video_url %>
-                      <% elsif lecture.status == 'attended_in_person' %>
-                        <% link_to "Open recording", lecture.recording_url %>
-                        <i class="fa fa-chalkboard-user fa-green-700"></i>
-                      <% elsif lecture.status == 'watched_recording' %>
-                        <i class="fa fa-video fa-green-700"></i>
-                      <% elsif lecture.status == 'missed' %>
-                        <button type="button" class="rounded bg-indigo-600 px-2 py-1 font-semibold text-white shadow-sm hover:bg-indigo-500 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-indigo-600">
-                          Start
-                        </button>
-                      <% else %>
-                        <% lecture.status %>
-                      <% end %>
-                    </td>
+                    <% if lecture.start_time.future? %>
+                      <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
+                      </td>
+
+                      <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
+                      </td>
+                    <% else %>
+                      <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
+                        <% if lecture.attendance.nil? %>
+                          <i class="fa fa-times text-red-700"></i>
+                        <% elsif lecture.attendance.kind == 'concurrent' %>
+                          <i class="fa fa-chalkboard-user fa-green-700"></i>
+                        <% elsif lecture.attendance.kind == 'catchup' %>
+                          <i class="fa fa-video fa-green-700"></i>
+                        <% else %>
+                          <% lecture.attendance.kind %>
+                        <% end %>
+                      </td>
+
+                      <td class="whitespace-nowrap px-3 py-4 text-sm text-gray-500">
+                        <% if lecture.attendance.nil? %>
+                          <button type="button" class="rounded bg-indigo-600 px-2 py-1 font-semibold text-white shadow-sm hover:bg-indigo-500 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-indigo-600">
+                            Start
+                          </button>
+                        <% else %>
+                          <% link_to "Open recording", lecture.recording_url %>
+                        <% end %>
+                      </td>
+                    <% end %>
                   </tr>
                 <% end %>
               <% end %>