Ability for Review Request emails to make use of the cc field when sending out emails to the Review Group.

Review Request #478 — Created July 29, 2008 and submitted — Latest diff uploaded

Tarun
Review Board SVN (deprecated)
reviewboard
A high priority for us was to differentiate between the Reviewer and non-reviewer when the review request emails were being sent out. This differentiation informs easily who the Reviewer is for a given Request. I have coded up the following to meet those requirements.

The attached patch adds the following properties:
In the emails that are sent out by Review Board, if there is a reviewer present( i.e someone mentioned in the people category) then the 'to' field will have the email of the Reviewer and the 'cc' field will consists of the other members present in the group(except the reviewer ...of course).
If no one is mentioned then the email will be sent out normally.


!!!Requirements:
The EmailMessage class in django (django/core/mail.py) does not make use of the 'cc' field. Therefore to make use of the 'cc' field, the following needs to be done:

1)Go to this link--> http://code.djangoproject.com/ticket/7722 <-- and download the mail.diff  and patch it to your django/core/maily.py . This allows for email messages to now use the 'cc' field. Once the patch has been applied a slight change needs to be done. The patch results in an indentation error at line 215. Once that has been adjusted, you are good to go!



---
I know this requires some additional changes to be made to the EmailMessage class.....but this is the best that can be done to use the 'cc' field.

************************************************************************************************************
UPDATE 2 : 9/2/2008

Instead of going through the above process where you have to apply a patch to Django, it is now possible for ReviewBoard to make use of the cc fields when sending out emails. To reiterate...there is no need to make any changes to Django the feature has been incorporated into ReviewBoard.
I have tested it on my Review Board setup and It works fine.

*******************************************************************************************************
UPDATE 2 : 9/2/2008

Tested on local ReviewBoard setup (Fedora 9)
Loading...