← Back to team overview

banking-addons-team team mailing list archive

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

 

@Stefan

I have changed the ownership of the branch so that you can push your changes to the clieop3 module directly here. It seems to me that you will just have to modify wizard/export_clieop.py line 304.

About the size of fields :
* in account_payment :
- communication : 64 
- communication2 : 64
* in account_banking_payment :
- communication2 is enlarged to 128 (I didn't change that ; it is present in the current stable branch)
* in account_banking_pain_base :
- communication is enlarged to 140 (because SEPA allows 140 char)

The SEPA modules only use the field "communication", they ignore the field "communication2". It has always been like that. I don't plan to change this, but it can easily be changed via an inherit of the method _prepare_field(), so those who really want to play with 2 fields can do it without problem.

-- 
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