[Tracker-discuss] [issue394] Better integration with Rietveld code review tool

Ezio Melotti metatracker at psf.upfronthosting.co.za
Thu Oct 6 17:19:22 CEST 2011


Ezio Melotti <ezio.melotti at gmail.com> added the comment:

The bit of code that sends the email is in rietveld/codereview/views.py:2540:_make_message and it's called by rietveld/codereview/views.py:2362:publish (note: the line numbers refer to r510, i.e. the version we are using).
A possible fix would be to:
 1) at line 2546 (after (actually, instead of) the part where the mail is sent to the cc), replace the body with something like "<user> submitted a review: <link>";
 2) send the mail to roundup;

It's also possible to create and add a new message directly to Roundup, but that's a little more involved.

_______________________________________________________
PSF Meta Tracker <metatracker at psf.upfronthosting.co.za>
<http://psf.upfronthosting.co.za/roundup/meta/issue394>
_______________________________________________________


More information about the Tracker-discuss mailing list