Code review comment for lp:~akretion-team/banking-addons/70-fully-handle-payment-types

Revision history for this message
Alexis de Lattre (alexis-via) wrote :

Can we really inherit the do_merge function ? It seems that this function is not designed to be inherited... I don't see how I could develop what you suggested (and it would be a bad idea to copy-paste the whole do_merge function inside the module account_payment_purchase). If you have an example of a module that inherit do_merge to add a field, please tell me.

« Back to merge proposal