Merge lp:~agilebg/openobject-addons/7.0-add-it-vat-22 into lp:openobject-addons/7.0
Proposed by
Leonardo Pistone
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Merged at revision: | 9501 | ||||||||
Proposed branch: | lp:~agilebg/openobject-addons/7.0-add-it-vat-22 | ||||||||
Merge into: | lp:openobject-addons/7.0 | ||||||||
Diff against target: |
85 lines (+42/-6) 2 files modified
l10n_it/data/account.tax.code.template.csv (+18/-0) l10n_it/data/account.tax.template.csv (+24/-6) |
||||||||
To merge this branch: | bzr merge lp:~agilebg/openobject-addons/7.0-add-it-vat-22 | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Lorenzo Battistini (community) | code review | Approve | |
Olivier Dony (Odoo) | Approve | ||
Review via email: mp+189282@code.launchpad.net |
Description of the change
Note: I made similar MPs for the other versions
The second commit adds a new Italian tax for 22%.
The first one is necessary to free codes 22a and 22b that were mistakenly used for a 2% tax.
Thanks!
To post a comment you must log in.
(Similar review to the 6.1 version)
I cannot vouch for the fiscal changes, but did the following sanity checks, for the record:
- this branch is also green on runbot in the agilebg team
- the module installs properly
- there were no references to the renamed entries, so none to correct elsewhere
- l10n_it is not translated so there is no translation to fix when renaming the IDs
- account/tax templates can be safely dropped/renamed 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 7.0 at revno 9501 revid:<email address hidden>
If the members of the Italian Community have any remark or patch, do not hesitate to comment here, it's not too late, even after the merge! :-)
Thanks!