In the description of the Review, all the developer put the similar template as shown below: What I will do is get the string between [Defect / Enhancement / New Feature] and [Problem] and populate as Email Subject line.
[Defect / Enhancement / New Feature] <MobC00087366 : Enhancement : Athena : Athenaray : Need to check in code of Athena audio cHAL and audio driver> [Problem] Need to fix the callbacks of audio manager. [Solution] Consolidated Audio mangager callbacks. Also fixed the pcm playback. [Reviewers] Liping Wang, Uday Gupta, Kuoyu Lin, Frank Shou On Nov 5, 12:26 pm, Christian Hammond <chip...@chipx86.com> wrote: > There was a bug filed for this > (http://code.google.com/p/reviewboard/issues/detail?id=521) but we decided > not to implement it, since in most cases you either 1) Will be posting to > mailman lists, which have their own subject formatting, or 2) Are using it > for filtering purposes, at which point the new X-ReviewRequest-* e-mail > headers are far more appropriate. > > You can certainly modify email.py, but know that this will make upgrades > hard, and the location of that file has changed in the 1.1 series. If you're > going to do this, read the page on maintaining Review Board forks > (http://www.reviewboard.org/docs/codebase/dev/custom-forks/). > > Also, if we were to allow subject modification, it would just be a prefix. > Kunjal, I'm not sure I understand how the regex or resulting subject line > would look, so maybe you can give me some examples? > > As for the .pyc files, it doesn't hurt to do that certainly. It *shouldn't* > be needed, though. The big thing is to restart the web server. > > Christian > > -- > Christian Hammond - chip...@chipx86.com > Review Board -http://www.reviewboard.org > VMware, Inc. -http://www.vmware.com > > > > On Thu, Nov 5, 2009 at 11:44 AM, Chris Clark <chris.cl...@ingres.com> wrote: > > > Kunjal wrote: > > > we want to customize the subject line > > > of the email which RB sends on submit of review. > > > Check out reviewboard/reviews/email.py - mail_review_request() look for > > "subject". > > > I "nice to have" would be a way to customize this from the site admin > > page. I'm not likely to implement this myself though :-( > > > Chris > > > > To unsubscribe from this group, send email to > > reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com> > > For more options, visit this group at > >http://groups.google.com/group/reviewboard?hl=en > > -~----------~----~----~----~------~----~------~--~---- Hide quoted text - > > - Show quoted text - --~--~---------~--~----~------------~-------~--~----~ Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~----------~----~----~----~------~----~------~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~----------~----~----~----~------~----~------~--~---
