From a44891b2d70e9deca48dcfe4e6ae7dee88cd29d7 Mon Sep 17 00:00:00 2001
From: Eric Amorde <eric.amorde@gmail.com>
Date: Sun, 15 Oct 2023 20:05:44 -0700
Subject: [PATCH] Improve navigation linking in manage views

---
 app/controllers/manage_controller.rb | 37 ++++++++++++++++++++++++++++
 app/views/manage/commits/show.erb    |  4 +--
 app/views/manage/pods/detail.erb     | 27 +++++++++++++++-----
 3 files changed, 60 insertions(+), 8 deletions(-)

diff --git a/app/controllers/manage_controller.rb b/app/controllers/manage_controller.rb
index 0e7d32b1..9fbc55ce 100644
--- a/app/controllers/manage_controller.rb
+++ b/app/controllers/manage_controller.rb
@@ -27,6 +27,43 @@ def self.hash_password(password)
       end
 
       helpers ManageHelper
+      helpers do
+        # @return [String]
+        #
+        def pod_url(pod_id)
+          "/manage/pods/#{pod_id}"
+        end
+
+        # @return [String]
+        #
+        def pod_link(pod)
+          "<a href=\"#{pod_url(pod.id)}\">#{pod.name}</a>"
+        end
+
+        # @return [String]
+        #
+        def commit_url(commit_id)
+          "/manage/commits/#{commit_id}"
+        end
+
+        # @return [String]
+        #
+        def commit_link(commit)
+          "<a href=\"#{commit_url(commit.id)}\">#{commit.id}</a>"
+        end
+
+        # @return [String]
+        #
+        def owner_url(owner_id)
+          "/manage/owners/#{owner_id}"
+        end
+
+        # @return [String]
+        #
+        def owner_link(owner)
+          "<a href=\"#{owner_url(owner.id)}\">#{owner.id}</a>"
+        end
+      end
 
       register Sinatra::Twitter::Bootstrap::Assets
 
diff --git a/app/views/manage/commits/show.erb b/app/views/manage/commits/show.erb
index 098e4d98..9de96805 100644
--- a/app/views/manage/commits/show.erb
+++ b/app/views/manage/commits/show.erb
@@ -1,14 +1,14 @@
 <% @title = @commit.pod_version.pod.name %>
 
 <div class="page-header">
-  <h1><%= @commit.pod_version.pod.name %> <small><%= @commit.pod_version.name %></small></h1>
+  <h1><%= pod_link(@commit.pod_version.pod) %> <small><%= @commit.pod_version.name %></small></h1>
 </div>
 
 <dl class="dl-horizontal">
   <dt>ID</dt>
   <dd><%= @commit.id %></dd>
   <dt>SHA</dt>
-  <dd><%= @commit.sha %><dd>
+  <dd><a href="<%= @commit.pod_version.data_url %>"><%= @commit.sha %></a><dd>
   <dt>Committer</dt>
   <dd><%= @commit.committer.public_attributes %></dd>
 </dl>
diff --git a/app/views/manage/pods/detail.erb b/app/views/manage/pods/detail.erb
index a7d0e4f9..08abcd3d 100644
--- a/app/views/manage/pods/detail.erb
+++ b/app/views/manage/pods/detail.erb
@@ -9,18 +9,31 @@
     <tr>
       <th>Version</th>
       <th>Commit</th>
+      <th>SHA</th>
+      <th>Published At</th>
     </tr>
   </thead>
   <tbody>
     <% @pod.versions.each do |version| %>
       <tr>
-        <td><%= version.name %></td>
         <td>
-          <% if version.published? %>
+          <%= version.name %>
+        </td>
+        <% if version.published? %>
+          <td>
+            <%= commit_link(version.last_published_by) %>
+          </td>
+          <td>
             <a href="<%= version.data_url %>"><%= version.commit_sha %></a>
-          <% else %>
-            Unpublished
-          <% end %>
+          </td>
+          <td>
+            <%= version.last_published_by.created_at.to_formatted_s(:rfc822) %>
+          </td>
+        <% else %>
+          <td>Unpublished</td>
+          <td></td>
+          <td></td>
+        <% end %>
         </td>
       </tr>
     <% end %>
@@ -41,7 +54,9 @@
   <tbody>
     <% @pod.owners.each do |owner| %>
       <tr>
-        <td><%= owner.id %></td>
+        <td>
+          <%= owner_link(owner) %>
+        </td>
         <td><%= owner.name %></td>
         <td><a href="mailto:<%= owner.email %>"><%= owner.email %></a></td>
       </tr>