Re: headers for review groups in emails

2008-11-06 Thread Henry Andrews
Sent: Friday, October 31, 2008 12:33:03 PM > Subject: Re: headers for review groups in emails > > > Cool. I'll probably do it next week as it's a change I did for work and I'm > off > today :-) > thanks, > -henry > > > > - Original Messa

Re: headers for review groups in emails

2008-10-31 Thread Henry Andrews
Cool. I'll probably do it next week as it's a change I did for work and I'm off today :-) thanks, -henry - Original Message > From: David Trowbridge <[EMAIL PROTECTED]> > To: reviewboard@googlegroups.com > Sent: Friday, October 31, 2008 11:27:03 AM >

Re: headers for review groups in emails

2008-10-31 Thread David Trowbridge
The best thing to do would be do post a review request on http://reviews.review-board.org/ If you want to add unit tests before posting the review, that would be awesome. I think this is only really documented as the "code reviews" link under "contributing" on the web site. We really need to wri

headers for review groups in emails

2008-10-31 Thread Henry Andrews
Hi folks, Along the lines of issue #521 (http://code.google.com/p/reviewboard/issues/detail?id=521) but less extensive and intrusive, I have a patch to set an email header (currently X-ReviewBoard-Groups) to a comma-separated lists of the target groups. This allows email filtering without mu