Review: Approve

Looks good, thanks for the update!

I cannot vouch for the fiscal changes, but did the following sanity checks, for 
the record:
 - the branch is green on runbot in the agilebg team
 - the module installs properly
 - there were no references to the renamed 22a->2a entries, so none to correct 
elsewhere
 - l10n_it is not translated so there is no translation to fix when renaming 
the IDs
 - account.templates can be safely dropped during module update because there 
is no connection between a template and the actual account after installation 
(at least not in the standard distribution)
 - Leonardo and Agile BG are part of OpenERP Italia community that is 
maintaining those modules.

Seeing the urgency of the fix, it was merged in addons 6.1 at revision 7255 
revid:[email protected]

If the members of the Italian Community have any remark or patch, do not 
hesitate to comment here, it's not too late! :-)
-- 
https://code.launchpad.net/~agilebg/openobject-addons/6.1-add-it-vat-22/+merge/189265
Your team OpenERP Italia core devs is requested to review the proposed merge of 
lp:~agilebg/openobject-addons/6.1-add-it-vat-22 into lp:openobject-addons/6.1.

_______________________________________________
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp

Rispondere a