Skip to content

[MIG] l10n_it_fatturapa_in_rc#3612

Merged
OCA-git-bot merged 17 commits into
OCA:16.0from
SirAionTech:16.0-mig-l10n_it_fatturapa_in_rc
Oct 13, 2023
Merged

[MIG] l10n_it_fatturapa_in_rc#3612
OCA-git-bot merged 17 commits into
OCA:16.0from
SirAionTech:16.0-mig-l10n_it_fatturapa_in_rc

Conversation

@SirAionTech

Copy link
Copy Markdown
Contributor

Sostituisce #3424.

sergiocorato and others added 16 commits September 27, 2023 12:41
[10.0] refactor in a separate module

[FIX] flake
ADD tests
FIX RC flag and inconsistency message about totals
…RC ITA" to be taken instead of "22% e-bill"

FIX

2019-10-28 16:27:59,292 5898 WARNING openerp_test odoo.addons.l10n_it_fatturapa_in.wizard.wizard_import_fatturapa: Too many taxes with percentage equals to '22.00'.
Fix it if required
2019-10-28 16:28:00,156 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: FAIL
2019-10-28 16:28:00,160 5898 INFO openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: ======================================================================
2019-10-28 16:28:00,160 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: FAIL: test_00_xml_import (odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc.TestInvoiceRC)
2019-10-28 16:28:00,160 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: Traceback (most recent call last):
2019-10-28 16:28:00,160 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: `   File "/home/travis/build/OCA/l10n-italy/l10n_it_fatturapa_in_rc/tests/test_fatturapa_in_rc.py", line 112, in test_00_xml_import
2019-10-28 16:28:00,160 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: `     '22% e-bill')
2019-10-28 16:28:00,160 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: ` AssertionError: 'Tax 22% Purchase RC ITA' != '22% e-bill'
2019-10-28 16:28:00,160 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: ` - Tax 22% Purchase RC ITA
2019-10-28 16:28:00,161 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc: ` + 22% e-bill
2019-10-28 16:28:00,161 5898 ERROR openerp_test odoo.addons.l10n_it_fatturapa_in_rc.tests.test_fatturapa_in_rc:
Currently translated at 100.0% (12 of 12 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_fatturapa_in_rc
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_fatturapa_in_rc/it/
@tafaRU

tafaRU commented Sep 27, 2023

Copy link
Copy Markdown
Member

/ocabot migration l10n_it_fatturapa_in_rc

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Sep 27, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 27, 2023
81 tasks
@SirAionTech SirAionTech marked this pull request as ready for review September 27, 2023 14:56
@SirAionTech

Copy link
Copy Markdown
Contributor Author

@odooNextev ho messo tutto in un commit di entrambi, fammi sapere cosa ne pensi

@odooNextev

Copy link
Copy Markdown
Contributor

@odooNextev ho messo tutto in un commit di entrambi, fammi sapere cosa ne pensi

Va bene

@SirAionTech SirAionTech marked this pull request as draft October 3, 2023 15:33
Co-authored-by: odooNextev <odoo@nextev.it>
@SirAionTech SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_in_rc branch from 7cc7997 to 0400985 Compare October 3, 2023 15:36
@SirAionTech SirAionTech marked this pull request as ready for review October 3, 2023 15:36
Comment thread l10n_it_fatturapa_in_rc/models/account_invoice.py
@SirAionTech SirAionTech requested a review from eLBati October 5, 2023 08:28
@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@tafaRU

tafaRU commented Oct 13, 2023

Copy link
Copy Markdown
Member

@MaurizioPellegrinet hai fatto la review funzionale?

@tafaRU

tafaRU commented Oct 13, 2023

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3612-by-tafaRU-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8e77de3 into OCA:16.0 Oct 13, 2023
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 1bf24ff. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.