openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~dreis-pt/contract-management/7.0-project_sla-dr into lp:contract-management

To: mp+199645@xxxxxxxxxxxxxxxxxx
From: Joël Grand-Guillaume @ camptocamp <joel.grandguillaume@xxxxxxxxxxxxxx>
Date: Thu, 19 Dec 2013 12:49:38 -0000
In-reply-to: <20131219113454.18457.38308.launchpad@ackee.canonical.com>
Reply-to: mp+199645@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review, no tests

Hi Daniel,

Thanks for submitting that here. A little tips: did you know you can click on "Resubmit Proposal" on the top right corner of an existing MP and change the source and destination branch ?

Using this, you conserve the historic of the discussion tha took place on the previous MP, even if source and destination branch are different. Very useful when you set a wrong destination branch or a wrong source branch or both !

Otherwise, LGTM, Thanks you.


-- 
https://code.launchpad.net/~dreis-pt/contract-management/7.0-project_sla-dr/+merge/199645
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:contract-management.

References

[Merge] lp:~dreis-pt/contract-management/7.0-project_sla-dr into lp:contract-management
From: Daniel Reis, 2013-12-19