banking-addons-team team mailing list archive

Thread
Date

Re: [Merge] lp:~banking-addons-drivers/banking-addons/70-communication-communication2-clarify into lp:banking-addons

To: Alexis de Lattre <alexis@xxxxxxxxxx>
From: Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>
Date: Tue, 25 Mar 2014 06:15:56 -0000
In-reply-to: <20140322010120.25578.53325.launchpad@ackee.canonical.com>
Reply-to: mp+212271@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review

Can you add 

AND communication is NULL

in the migrate query?

To avoid overwriting communication for people using account_banking_payment_export without account_banking_payment.
-- 
https://code.launchpad.net/~banking-addons-drivers/banking-addons/70-communication-communication2-clarify/+merge/212271
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.

References

[Merge] lp:~akretion-team/banking-addons/70-communication-communication2-clarify into lp:banking-addons
From: Alexis de Lattre, 2014-03-22