Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

14.0 fix l10n it fatturapa in mig scripts #4

Open
wants to merge 14 commits into
base: 14.0
Choose a base branch
from

Conversation

SimoneVagile
Copy link
Owner

migration from version 8.0

vincenzoterzulli and others added 13 commits June 28, 2021 11:50
…ther backports

* [FIX] l10n_it_fatturapa: withholding tax invoice received - e-invoice 1.6 (OCA#1878)
* [FIX] l10n_it_fatturapa_out_wt: pep8 fixes
* [IMP] l10n_it_fatturapa_in: migration script
* [FIX] l10n_it_fatturapa_out_wt: Fixed ordering in DatiRitenuta generation, according to XML test file.
…CA#1878)

* [12.0][FIX] fatturapa withholding tax invoice received - e-invoice 1.6

* [FIX] l10n_it_fatturapa_out_wt pep8 fixes

This reverts commit 421ef9e51634383263638e9b52ea4038829f62ec.

* [IMP] migration script

* [FIX] l10n_it_fatturapa_out_wt: Fixed ordering in DatiRitenuta generation, according to XML test file.
Otherwise:
2020-09-17 07:34:14,789 7808 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_wt.tests.test_fatturapa_wt: ` AssertionError: b'<ns[1677 chars]a>RT02</TipoRitenuta><ImportoRitenuta>2.00</Im[1155 chars]ica>' != b'<ns[1677 chars]a>RT04</TipoRitenuta><ImportoRitenuta>0.83</Im[1155 chars]ica>'

Co-authored-by: SimoRubi <[email protected]>
Copy link
Collaborator

@fredzamoabg fredzamoabg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok ma matti a posto pre-commit please

@SimoneVagile SimoneVagile force-pushed the 14.0-fix-l10n_it_fatturapa_in-mig-scripts branch from c8b97cc to 8bbca75 Compare July 8, 2021 08:18
@SimoneVagile
Copy link
Owner Author

ok ma matti a posto pre-commit please

fatto, grazie

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants