You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2019/02/14 13:19:29 UTC

[whimsy] branch master updated: Add legend, and distinguish members/committers

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new e96b637  Add legend, and distinguish members/committers
e96b637 is described below

commit e96b637f0218bb0da34a09df2458f8ed496e2dd8
Author: Sebb <se...@apache.org>
AuthorDate: Thu Feb 14 13:19:27 2019 +0000

    Add legend, and distinguish members/committers
---
 www/members/security-subs.cgi | 34 +++++++++++++++++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

diff --git a/www/members/security-subs.cgi b/www/members/security-subs.cgi
index 9d9a241..c7e0ebb 100755
--- a/www/members/security-subs.cgi
+++ b/www/members/security-subs.cgi
@@ -39,6 +39,36 @@ _html do
       end
 
     elsif lists[path]
+      _table do
+        _tr do
+          _th 'Legend'
+        end
+        _tr do
+          _td do
+            _ 'Archiver'
+          end
+        end
+        _tr do
+          _td class: 'bg-success' do
+            _ 'ASF member or project member'
+          end
+        end
+        _tr do
+          _td class: 'bg-info' do
+            _ 'Project committer - not on (P)PMC'
+          end
+        end
+        _tr do
+          _td class: 'bg-warning' do
+            _ 'ASF committer not associated with the project'
+          end
+        end
+        _tr do
+          _td class: 'bg-danger' do
+            _ 'Person (email) not recognised'
+          end
+        end
+      end
       podling = ASF::Podling.find(path)
       committee = ASF::Committee.find(path)
       project = ASF::Project.find(path)
@@ -64,8 +94,10 @@ _html do
           lists[path].sort_by {|email| email.downcase}.each do |email|
             person = ASF::Person.find_by_email(email)
             if person
-              if person.asf_member? or project.members.include? person
+              if person.asf_member? or project.owners.include? person
                 color = 'bg-success'
+              elsif project.members.include? person
+                color = 'bg-info'
               else
                 color = 'bg-warning'
               end