From 5edfaf5a8ac746f534a41500e75d2673cfbada09 Mon Sep 17 00:00:00 2001
From: Igor Moyzeson <igormoyzeson@berkeley.edu>
Date: Tue, 7 Apr 2026 00:42:28 -0700
Subject: [PATCH] updated requests dashboard

---
 app/views/requests/instructor_index.html.erb | 19 +++++++++++++------
 app/views/requests/student_index.html.erb    |  6 ++++++
 2 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/app/views/requests/instructor_index.html.erb b/app/views/requests/instructor_index.html.erb
index 4274f8ef..57e440ed 100644
--- a/app/views/requests/instructor_index.html.erb
+++ b/app/views/requests/instructor_index.html.erb
@@ -46,7 +46,8 @@
                 <th class="text-center align-content-center" style="min-width: 198px;" data-priority="8">Original Due Date</th>
                 <th class="text-center align-content-center" style="min-width: 198px;" data-priority="4">Requested Due Date</th>
                 <th class="text-center align-content-center" style="min-width: 90px;" data-priority="6"># of Days</th>
-                <th class="text-center align-content-center" style="min-width: 200px;" data-priority="3">Status</th>
+                <th class="text-center align-content-center" style="min-width: 110px;" data-priority="3">Status</th>
+                <th class="text-center align-content-center" style="min-width: 200px;" data-priority="4">Updated At</th>
               </tr>
             </thead>
             <tbody>
@@ -127,17 +128,23 @@
                           <% else %>
                             <span class="badge text-bg-success">Approved</span>
                           <% end %>
-                          on <%= request.updated_at.strftime('%a, %b %-d at %-I:%M%P') || 'N/A' %>
-                          by <%= request.auto_approved ? 'Auto Approval System' : request.last_processed_by_user&.name || 'Unknown' %>
                         <% when 'denied' %>
-                          <span class="badge text-bg-danger me-3">Denied</span>
-                          on <%= request.updated_at.strftime('%a, %b %-d at %-I:%M%P') || 'N/A' %>
-                          by <%= request.last_processed_by_user&.name || 'Unknown' %>
+                          <span class="badge text-bg-danger">Denied</span>
                         <% else %>
                           <span class="badge text-bg-secondary">Unknown</span>
                       <% end %>
                     <% end %>
                   </td>
+                  <td class="text-center align-content-center" data-order="<%= request.status == 'pending' ? '' : request.updated_at.iso8601 %>">
+                    <% case request.status %>
+                      <% when 'approved' %>
+                        <%= request.updated_at.strftime('%a, %b %-d at %-I:%M%P') %>
+                        by <%= request.auto_approved ? 'Auto Approval System' : request.last_processed_by_user&.name || 'Unknown' %>
+                      <% when 'denied' %>
+                        <%= request.updated_at.strftime('%a, %b %-d at %-I:%M%P') %>
+                        by <%= request.last_processed_by_user&.name || 'Unknown' %>
+                    <% end %>
+                  </td>
                 </tr>
               <% end %>
             </tbody>
diff --git a/app/views/requests/student_index.html.erb b/app/views/requests/student_index.html.erb
index e955f237..4ed64ef4 100644
--- a/app/views/requests/student_index.html.erb
+++ b/app/views/requests/student_index.html.erb
@@ -14,6 +14,7 @@
                 <th class="text-center align-content-center" style="min-width: 198px;">Requested Due Date</th>
                 <th class="text-center align-content-center" style="min-width: 90px;"># of Days</th>
                 <th class="text-center align-content-center">Status</th>
+                <th class="text-center align-content-center" style="min-width: 198px;">Updated At</th>
                 <th class="text-center align-content-center" style="min-width: 125px;">Actions</th>
                 </tr>
             </thead>
@@ -56,6 +57,11 @@
                         <span class="badge text-bg-secondary">Unknown</span>
                       <% end %>
                     </td>
+                    <td class="text-center align-content-center">
+                      <% if request.status == 'approved' || request.status == 'denied' %>
+                        <%= request.updated_at.strftime('%a, %b %-d at %-I:%M%P') %>
+                      <% end %>
+                    </td>
                     <td class="text-center align-content-center">
                     <%= link_to 'View', course_request_path(@course, request), class: "btn btn-sm btn-primary" %>
                     <% if request&.status == 'pending' %>