openerp-community-reviewer team mailing list archive

["Frederic Clementi - Camptocamp.com" <frederic.clementi@xxxxxxxxxxxxxx>]

Review: Needs Fixing functionnal

3 points to be corrected on my side : 

1 - If in  the wizard, Duplicate taxe code linked is ticked, tax codes are not duplicated on the new taxe on the refund side (ref_base_code_id & ref_tax_code_id)... still refering to the old tax codes.


2 - On the wizard form (account_tax_update_config), the field name should be clearly named : 'legacy taxes prefix' (or a better description if you find an other one)

3 - You have possibility to edit taget taxe from the wizard form... but change the name, rate... you will not see new name on the wizard form, which is a shame and can lead to misunderstanding.






-- 
https://code.launchpad.net/~camptocamp/account-financial-tools/port_of_trp_update_tax/+merge/194348
Your team Account Core Editors is requested to review the proposed merge of lp:~camptocamp/account-financial-tools/port_of_trp_update_tax into lp:account-financial-tools.