Also, it would really help if you could make some unit tests for this? Make
sure the diff revision is what we expect in diffs and parent diffs.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Mon, Jun 27, 2011 at 12:59 AM, Christian Hammond <chip...@chipx86.com>wrote:

> Hi,
>
> Thanks for bringing that again to my attention.
>
> Yeah, I think we can get this in. I'll run some tests on it and then push
> it.
>
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Mon, Jun 27, 2011 at 12:34 AM, Clemens Ballarin <balla...@aicas.com>wrote:
>
>> Hi Christian,
>>
>> I wonder whether my patch
>>
>>  Fix display of revision if there is a parent diff.
>>  
>> http://reviews.reviewboard.**org/r/2094/<http://reviews.reviewboard.org/r/2094/>
>>
>> will make it into Release 1.6.  My understanding was that it will.  Are
>> you still planning to incorporate it?
>>
>> Clemens
>>
>>
>>
>> On 23 Jun 2011, at 10:48, Christian Hammond wrote:
>>
>>  Hi everyone,
>>>
>>> We released Review Board 1.6 RC1 tonight, which has several new features,
>>> fixes, and performance enhancements. We're very close to a final release, so
>>> let us know if you hit any problems.
>>>
>>> For more information on the release, see http://www.reviewboard.org/**
>>> news/2011/06/23/review-board-**16-rc1-released/<http://www.reviewboard.org/news/2011/06/23/review-board-16-rc1-released/>
>>>
>>> Christian
>>>
>>> --
>>> Christian Hammond - chip...@chipx86.com
>>> Review Board - http://www.reviewboard.org
>>> VMware, Inc. - http://www.vmware.com
>>>
>>> --
>>> Want to help the Review Board project? Donate today at
>>> http://www.reviewboard.org/**donate/<http://www.reviewboard.org/donate/>
>>> Happy user? Let us know at 
>>> http://www.reviewboard.org/**users/<http://www.reviewboard.org/users/>
>>> -~----------~----~----~----~--**----~----~------~--~---
>>> To unsubscribe from this group, send email to reviewboard+unsubscribe@**
>>> googlegroups.com <reviewboard%2bunsubscr...@googlegroups.com>
>>> For more options, visit this group at http://groups.google.com/**
>>> group/reviewboard?hl=en<http://groups.google.com/group/reviewboard?hl=en>
>>>
>>
>> --
>> Dr. Clemens Ballarin -- Senior Software Engineer
>>
>> aicas GmbH  *  Haid-und-Neu-Straße 18  *  D-76131 Karlsruhe  *  Germany
>> http://www.aicas.com  *  phone +49 721 663 968-35  *  fax -99
>>
>> USt-Id: DE216375633, Handelsregister HRB 109481, AG Mannheim
>> Geschäftsführer: Dr. James J. Hunt
>>
>>
>> --
>> Want to help the Review Board project? Donate today at
>> http://www.reviewboard.org/**donate/ <http://www.reviewboard.org/donate/>
>> Happy user? Let us know at 
>> http://www.reviewboard.org/**users/<http://www.reviewboard.org/users/>
>> -~----------~----~----~----~--**----~----~------~--~---
>> To unsubscribe from this group, send email to reviewboard+unsubscribe@**
>> googlegroups.com <reviewboard%2bunsubscr...@googlegroups.com>
>> For more options, visit this group at http://groups.google.com/**
>> group/reviewboard?hl=en<http://groups.google.com/group/reviewboard?hl=en>
>>
>
>

-- 
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

Reply via email to