Partners can view request comment and sender (#5160)

* Partner dashboard: add 'comment and sender' to Requests in Progress

* Comment is truncated to 20 chars; rest displayed via tooltip

* Partner requests: add 'comment and sender' to Request History

* Add specs

* Refactor script for tooltip

Author: jp524

app/views/partners/dashboards/_requests_in_progress.html.erb

@@ -7,6 +7,7 @@
         <th scope="col" class="p-4" style='width:150px'>Request Date</th>
         <th scope="col" class="p-4">Number of Items Requested</th>
         <th scope="col" class="p-4">Items Requested</th>
+        <th scope="col" class="p-4">Comment and Sender</th>
       </tr>
     </thead>
     <tbody>
@@ -33,8 +34,24 @@
               </span>
             <% end %>
           </td>
+          <td class="p-4">
+            <% comment = request.comments %>
+            <% if comment.present? %>
+              <div class="mb-2" data-bs-toggle="tooltip" data-bs-title="<%= comment %>">
+                <%= truncate(comment, length: 20) %>
+              </div>
+            <% end %>
+            <a href="mailto:<%= request.requester.email %>"><%= request.requester.email %></a>
+          </td>
         </tr>
       <% end %>
     </tbody>
   </table>
 </section>
+
+<script type="module">
+  $(document).ready(function() {
+    const tooltipTriggerList = document.querySelectorAll('[data-bs-toggle="tooltip"]');
+    [...tooltipTriggerList].forEach(tooltipTriggerEl => new bootstrap.Tooltip(tooltipTriggerEl));
+  })
+</script>

app/views/partners/requests/_history.html.erb

@@ -5,7 +5,9 @@
     <thead>
       <tr class="border-bottom border-dark">
         <th scope="col" class="p-4 w-40">Request Date</th>
+        <th scope="col" class="p-4">Number of Items Requested</th>
         <th scope="col" class="p-4">Items Requested</th>
+        <th scope="col" class="p-4">Comment and Sender</th>
       </tr>
     </thead>
     <tbody>
@@ -23,8 +25,24 @@
               </span>
             <% end %>
           </td>
+          <td class="p-4">
+            <% comment = partner_request.comments %>
+            <% if comment.present? %>
+              <div class="mb-2" data-bs-toggle="tooltip" data-bs-title="<%= comment %>">
+                <%= truncate(comment, length: 20) %>
+              </div>
+            <% end %>
+            <a href="mailto:<%= partner_request.requester.email %>"><%= partner_request.requester.email %></a>
+          </td>
         </tr>
       <% end %>
     </tbody>
   </table>
 </section>
+
+<script type="module">
+  $(document).ready(function() {
+    const tooltipTriggerList = document.querySelectorAll('[data-bs-toggle="tooltip"]');
+    [...tooltipTriggerList].forEach(tooltipTriggerEl => new bootstrap.Tooltip(tooltipTriggerEl));
+  })
+</script>

spec/requests/partners/dashboard_requests_spec.rb

@@ -59,6 +59,16 @@
 
       expect(response.body).to match(/7\s+#{item1.name}/m)
     end
+
+    it "displays comment and sender" do
+      request = create(:request, :pending, partner:, request_items: [])
+      create(:item_request, request:, quantity: 16, item: item1)
+
+      get partners_dashboard_path
+
+      expect(response.body).to include(request.comments)
+      expect(response.body).to include(request.requester.email)
+    end
   end
 
   it "displays upcoming distributions" do

spec/requests/partners/requests_spec.rb

@@ -30,6 +30,13 @@
       subject.call
       expect(response.body).to include("684")
     end
+
+    it "displays comment and sender" do
+      request = create(:request, partner_id: partner.id, request_items: [{item_id: item1.id, quantity: '125'}])
+      subject.call
+      expect(response.body).to include(request.comments)
+      expect(response.body).to include(request.requester.email)
+    end
   end
 
   describe "GET #new" do