This patch implements the changes that will open the roster/committer page in a 
different window in the secretary workbench.

diff --git a/www/secretary/workbench/server.rb 
b/www/secretary/workbench/server.rb
index 827225db..37fab606 100644
--- a/www/secretary/workbench/server.rb
+++ b/www/secretary/workbench/server.rb
@@ -77,7 +77,7 @@ get '/' do
       href: "/roster/committer/#{id}",
       from: ASF::Person.find(id).cn,
       subject: "Pending emeritus request - #{days.to_s} days old",
-      status: days < 10.0 ? 'Pending' : 'Ready'
+      status: days < 10.0 ? :emeritusPending : :emeritusReady
     }
   end
 
diff --git a/www/secretary/workbench/views/index.js.rb 
b/www/secretary/workbench/views/index.js.rb
index c4ff9351..0e64d2e6 100644
--- a/www/secretary/workbench/views/index.js.rb
+++ b/www/secretary/workbench/views/index.js.rb
@@ -46,8 +46,9 @@ class Index < Vue
             }
 
             _tr row_options do
+              target = ((message.status == :emeritusReady) || (message.status 
== :emeritusPending)) ? "_blank" : "_self"
               _td do
-                _a time, href: "#{message.href}", title: message.time
+                _a time, href: "#{message.href}", title: message.time, target: 
target
               end 
               _td message.from
               _td message.subject

Craig L Russell
c...@apache.org

Reply via email to