sebbASF commented on code in PR #179:
URL: https://github.com/apache/whimsy/pull/179#discussion_r1060138374


##########
www/secretary/workbench/templates/resubmit.erb:
##########
@@ -1,10 +1,14 @@
 Dear <%= @email.display_names.join(', ') %>,
 
 Thanks for the ICLA, however we already have a copy.
-The <%= @cttee %> can use the existing ICLA to request your account by using 
Whimsy:
+If you are not already a committer, the <%= @cttee %> can use the existing 
ICLA to request your account by using Whimsy:
 
 https://whimsy.apache.org/officers/acreq.cgi (PMC chair and ASF members only)
 
+If you are already a committer on another project, the <%= @cttee %> can use 
Whimsy to give you permissions to the repository:

Review Comment:
   What about users who are not currently on another project?
   The phrase "on another project" is not necessary and should be dropped.



##########
www/secretary/workbench/templates/resubmit.erb:
##########
@@ -1,10 +1,14 @@
 Dear <%= @email.display_names.join(', ') %>,
 
 Thanks for the ICLA, however we already have a copy.
-The <%= @cttee %> can use the existing ICLA to request your account by using 
Whimsy:
+If you are not already a committer, the <%= @cttee %> can use the existing 
ICLA to request your account by using Whimsy:
 
 https://whimsy.apache.org/officers/acreq.cgi (PMC chair and ASF members only)
 
+If you are already a committer on another project, the <%= @cttee %> can use 
Whimsy to give you permissions to the repository:
+
+https://whimsy.apache.org/roster/committee
+

Review Comment:
   There's a bit missing here



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@whimsical.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to