Re: Issue 2483 in reviewboard: Minor tweak to review request email notification template to include repository name along with review group

2013-02-04 Thread reviewboard

Updates:
Status: Fixed

Comment #4 on issue 2483 by trowb...@gmail.com: Minor tweak to review  
request email notification template to include repository name along with  
review group

http://code.google.com/p/reviewboard/issues/detail?id=2483

Fixed in release-1.7.x (daf2016). Thanks!

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Issue 2483 in reviewboard: Minor tweak to review request email notification template to include repository name along with review group

2013-01-18 Thread reviewboard


Comment #3 on issue 2483 by kat.sch...@gmail.com: Minor tweak to review  
request email notification template to include repository name along with  
review group

http://code.google.com/p/reviewboard/issues/detail?id=2483

I call dibs!

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2483 in reviewboard: Minor tweak to review request email notification template to include repository name along with review group

2013-01-12 Thread reviewboard

Updates:
Labels: EasyFix

Comment #2 on issue 2483 by trowb...@gmail.com: Minor tweak to review  
request email notification template to include repository name along with  
review group

http://code.google.com/p/reviewboard/issues/detail?id=2483

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2483 in reviewboard: Minor tweak to review request email notification template to include repository name along with review group

2012-12-23 Thread reviewboard

Updates:
Labels: Component-EMail

Comment #1 on issue 2483 by trowb...@gmail.com: Minor tweak to review  
request email notification template to include repository name along with  
review group

http://code.google.com/p/reviewboard/issues/detail?id=2483

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Issue 2483 in reviewboard: Minor tweak to review request email notification template to include repository name along with review group

2012-02-10 Thread reviewboard

Status: New
Owner: 
Labels: Type-Enhancement Priority-Medium

New issue 2483 by gaz...@gmail.com: Minor tweak to review request email  
notification template to include repository name along with review group

http://code.google.com/p/reviewboard/issues/detail?id=2483

Version:  1.6.3
OS: RHEL 5.7
Browser:  Chrome, Firefox  (not relevant in this case)


I patched the  
reviewboard/templates/notifications/review_request_email.{html,txt} files  
in our deployment to include the related repository name, such as in the  
following line:


  Review request for {% reviewer_list review_request %}  
({{review_request.repository.name}}).


This is useful for anyone who ever shares review groups across multiple  
repositories.  In our case, we have several dozen related small repos that  
all share a single review group, and we found it often difficult to  
differentiate one from another in the review request emails.  We would  
previously need to actually view the review request in order to find the  
repository name.  I realize that creating/managing distinctly named review  
groups for every repository would avoid needing to add this info to the  
notification, but we have found it simpler to maintain a smaller number of  
review groups.


I found a rejected enhancement request for more formal template  
customization support.  So, I simply modified the template files directly  
and thought I'd submit this suggestion for the minor tweak so that I won't  
need to patch after each upgrade (and also because I suspect this might be  
generally useful).


--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.