Merge lp:~camptocamp/ocb-addons/ocb-7.0-fix-1108228-hip-afe into lp:ocb-addons

Proposed by Alexandre Fayolle - camptocamp
Status: Merged
Merged at revision: 9579
Proposed branch: lp:~camptocamp/ocb-addons/ocb-7.0-fix-1108228-hip-afe
Merge into: lp:ocb-addons
Diff against target: 19 lines (+10/-0)
1 file modified
stock/wizard/stock_invoice_onshipping_view.xml (+10/-0)
To merge this branch: bzr merge lp:~camptocamp/ocb-addons/ocb-7.0-fix-1108228-hip-afe
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Stefan Rijnhart (Opener) Approve
Christophe CHAUVET code review, not test Approve
Review via email: mp+188010@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

this is used in production

Revision history for this message
Christophe CHAUVET (christophe-chauvet) wrote :

Hi Alexandre

It's ok for me, ne a difficult patch to review

Regards,

review: Approve (code review, not test)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks! I checked that its the same as the original patch by the OpenERP dev on trunk (and that it makes sense).

Email address of the original author is not recognized by Launchpad, maybe it's a tinyerp address. Could be checked out at merge time.

review: Approve
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Holger,

thanks for the merge. At merge time, you don't have to specify the --fixes argument. This is already included in the merged revisions. It also tells Launchpad that the series branch fixes the bug, which is technically correct but not informative. The --fixes argument should be applied to the bug fixing branch only.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'stock/wizard/stock_invoice_onshipping_view.xml'
--- stock/wizard/stock_invoice_onshipping_view.xml 2012-10-23 16:05:04 +0000
+++ stock/wizard/stock_invoice_onshipping_view.xml 2013-09-27 10:40:01 +0000
@@ -40,5 +40,15 @@
40 view_type="form"40 view_type="form"
41 target="new"41 target="new"
42 id="action_stock_invoice_onshipping"/>42 id="action_stock_invoice_onshipping"/>
43
44 <act_window name="Create Invoices"
45 res_model="stock.invoice.onshipping"
46 src_model="stock.picking"
47 key2="client_action_multi"
48 multi="True"
49 view_mode="form"
50 view_type="form"
51 target="new"
52 id="action_stock_invoice_onshipping"/>
43 </data>53 </data>
44</openerp>54</openerp>