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.

Sign up for GitHub

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

Jump to bottom

[MIG] l10n_it_fatturapa_in_rc: Migration to 14.0 #2149

Merged
merged 10 commits into from
Mar 3, 2022
Merged

[MIG] l10n_it_fatturapa_in_rc: Migration to 14.0 #2149

merged 10 commits into from
Mar 3, 2022

Conversation

TheMule71
Copy link
Contributor

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@TheMule71 TheMule71 mentioned this pull request Mar 4, 2021
Open
76 tasks
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Mar 5, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
1f862f7
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from d05c914 to f69b1c2 Compare May 28, 2021 17:17
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
d3db81e
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from f69b1c2 to a77dc7d Compare June 9, 2021 09:02
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from a77dc7d to 7a2a5ff Compare June 18, 2021 14:34
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
f7c3c7b
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
bbada5d
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
8383326
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
925dbb6
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch 2 times, most recently from 855665b to 62a0a17 Compare July 2, 2021 12:27
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
3a90bfb
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
eed03c4
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
ed48501
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
6360459
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from 62a0a17 to be56071 Compare July 17, 2021 13:47
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
60aab23
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
0787332
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
62ecd03
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
997f53c
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 6, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
ed1164e
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 20, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
26093ab
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
94f7770
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
7dc2548
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch 2 times, most recently from fd36e59 to 2e2ab0e Compare September 30, 2021 07:34
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
f759a52
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
a94e299
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
d91df80
eLBati and others added 7 commits February 4, 2022 10:58
@eLBati @TheMule71
FIX l10n_it_fatturapa_in_rc when importing e-invoice with Minimum det…
5f45c3c 
…ail level
@eLBati @TheMule71
IMP l10n_it_fatturapa_in_rc automated tests: avoid "Tax 22% Purchase …
988d4dd 
…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:
@tafaRU @TheMule71
l10n_it_fatturapa_in_rc: fix auto_install key
254d358
@TheMule71
merge PR 1264 1717 1728 1737 1739
b20d10b
@eLBati @TheMule71
IMP l10n_it_fatturapa_in_rc: handling new RC codes from 1.6 version
0a34f33
@sergiocorato @TheMule71
[FIX] demo nature data remove no more selectable
e3dc030
@TheMule71
[IMP] l10n_it_fatturapa_in_rc: black, isort, prettier
4ddaee3
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from 48927c3 to 9a851ce Compare February 4, 2022 09:58
@tafaRU
Copy link
Member

tafaRU commented Feb 4, 2022

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from 9a851ce to d889f51 Compare February 4, 2022 14:20
@tafaRU
Copy link
Member

tafaRU commented Feb 4, 2022

@TheMule71 falliscono i test in https://github.com/OCA/l10n-italy/runs/5067772639?check_suite_focus=true#step:8:782

@TheMule71
Copy link
Contributor Author

Aspe, questa non era veramente pronta, ci stavo lavorando...

@tafaRU
Copy link
Member

tafaRU commented Feb 4, 2022

Aspe, questa non era veramente pronta, ci stavo lavorando...

ok, stamattina in call avevo capito diversamente, piuttosto allora mettila in draft 😉

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_in_rc branch from d889f51 to 3511f62 Compare February 4, 2022 16:16
@TheMule71
Copy link
Contributor Author

TheMule71 commented Feb 4, 2022 •
edited
Loading

In effetti no, era a posto e ha smesso di essere posto da sola. Comunque vediamo se è verde adesso. Queste le differenze:

diff --git a/l10n_it_fatturapa_in_rc/tests/test_fatturapa_in_rc.py b/l10n_it_fatturapa_in_rc/tests/test_fatturapa_in_rc.py
index 08c2e1b37..075ac59b4 100644
--- a/l10n_it_fatturapa_in_rc/tests/test_fatturapa_in_rc.py
+++ b/l10n_it_fatturapa_in_rc/tests/test_fatturapa_in_rc.py
@@ -123,7 +123,7 @@ class TestInvoiceRC(FatturapaCommon):
         self.assertEqual(invoice.get_tax_amount_added_for_rc(), 4.4)
         self.assertEqual(invoice.amount_tax, 5.5)
         self.assertEqual(invoice.e_invoice_amount_tax, 1.1)
-        invoice._post()
+        invoice.action_post()
         self.assertAlmostEqual(invoice.rc_self_invoice_id.amount_total, 24.4)
         self.assertEqual(invoice.rc_self_invoice_id.amount_tax, 4.4)
         self.assertEqual(invoice.rc_self_invoice_id.amount_untaxed, 20)

@andreampiovesana
Copy link
Contributor

merge?

@tafaRU
Copy link
Member

tafaRU commented Mar 3, 2022

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2149-by-tafaRU-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b59ecd8 into OCA:14.0 Mar 3, 2022
@OCA-git-bot
Copy link
Contributor

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

@TheMule71 TheMule71 deleted the 14.0-mig-l10n_it_fatturapa_in_rc branch June 3, 2022 09:04
@TheMule71 TheMule71 restored the 14.0-mig-l10n_it_fatturapa_in_rc branch June 3, 2022 09:16
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
@TheMule71
add OCA#2077 OCA#2149 OCA#2151
8c610f6
@TheMule71 TheMule71 deleted the 14.0-mig-l10n_it_fatturapa_in_rc branch February 3, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SimoRubi SimoRubi SimoRubi approved these changes

Assignees
No one assigned
Labels
merged 🎉 migration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@TheMule71 @tafaRU @OCA-git-bot @andreampiovesana @SimoRubi @francesco-ooops @sergiocorato @eLBati