Code review comment for lp:~camptocamp/openerp-rma/7.0-crm_claim_rma-fix-lp1317045_rde

Revision history for this message
Romain Deheele - Camptocamp (romaindeheele) wrote :

Hello Benoit,

FYI, the return address remains available on the field partner_id on... stock.move object :(
the help on this field describes :"Optional address where goods are to be delivered, specifically used for allotment""
and crm_claim_rma forces you to have the same address on lines used to generate the return picking.
On my side, I added a new report on stock.picking to print a return picking (because it was very specific : landscape and other details).
but, in the generic case, you point out a case that deserves to be clearer : should we use the standard picking report or one new report?
Joel, what do you think about it?

Thanks,

Romain

« Back to merge proposal