Skip to content

[16.0][MIG] l10n_it_intrastat#3069

Merged
OCA-git-bot merged 41 commits into
OCA:16.0from
Borruso:16.0-mig-l10n_it_intrastat
Sep 22, 2023
Merged

[16.0][MIG] l10n_it_intrastat#3069
OCA-git-bot merged 41 commits into
OCA:16.0from
Borruso:16.0-mig-l10n_it_intrastat

Conversation

@Borruso

@Borruso Borruso commented Nov 25, 2022

Copy link
Copy Markdown
Contributor

Migration l10n_it_intrastat from 14.0 to 16.0
eliminata dipendenza da web_widget_char_size

--
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

@Borruso Borruso mentioned this pull request Nov 25, 2022
81 tasks
@TheMule71

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_it_intrastat

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Dec 2, 2022
@Borruso Borruso force-pushed the 16.0-mig-l10n_it_intrastat branch 2 times, most recently from f3c39aa to 9bdae10 Compare May 25, 2023 08:00
@TheMule71

Copy link
Copy Markdown
Contributor

/ocabot rebase

Lara Baggio and others added 23 commits September 14, 2023 19:47
Currently translated at 0.4% (42 of 10990 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 14.5% (1595 of 10982 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 14.5% (1595 of 10982 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Investigating an issue reported by Sergio Zanchetta.
Currently translated at 14.5% (1593 of 10990 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 14.6% (1601 of 10988 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/it/
Currently translated at 0.5% (56 of 10988 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat/es/
…n't adapt type 'account.intrastat.transaction.nature' + test
Pre-commit issues:
l10n_it_intrastat/models/account.py:304:5: C901 'AccountInvoice.compute_intrastat_lines' is too complex (17)
l10n_it_intrastat/models/account.py:540: [C8108(method-compute), AccountInvoiceIntrastat] Name of compute method should start with "_compute_"
l10n_it_intrastat/models/account.py:170:96: B950 line too long (95 > 80 characters)
l10n_it_intrastat/models/account.py:176:100: B950 line too long (99 > 80 characters)
l10n_it_intrastat/models/account.py:189:92: B950 line too long (91 > 80 characters)
l10n_it_intrastat/models/account.py:303:43: E741 ambiguous variable name 'l'
l10n_it_intrastat/models/account.py:373:21: B007 Loop control variable 'key' not used within the loop body. If this is intended, start the name with an underscore.
l10n_it_intrastat/models/account.py:380:25: B007 Loop control variable 'key' not used within the loop body. If this is intended, start the name with an underscore.
l10n_it_intrastat/models/account.py:397:17: B007 Loop control variable 'key' not used within the loop body. If this is intended, start the name with an underscore.
l10n_it_intrastat/models/account.py:409:13: E741 ambiguous variable name 'l'
l10n_it_intrastat/tests/test_intrastat.py:74:35: E741 ambiguous variable name 'l'

Migrate old invoice fields and fix tests
SicurSam and others added 14 commits September 14, 2023 19:48
Currently translated at 14.5% (1603 of 10988 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
Currently translated at 100.0% (10995 of 10995 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
…t on partner change. Better intrastat handling when invoice created from po,so.
Currently translated at 99.9% (10994 of 10995 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
Currently translated at 99.9% (10994 of 10995 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
Currently translated at 100.0% (10995 of 10995 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_intrastat
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat/it/
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, PR rebased to 16.0.

@OCA-git-bot OCA-git-bot force-pushed the 16.0-mig-l10n_it_intrastat branch from 9bdae10 to 3e3d1df Compare September 14, 2023 19:48

@TheMule71 TheMule71 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@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

@TheMule71

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-3069-by-TheMule71-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 22, 2023
Signed-off-by TheMule71
@OCA-git-bot

Copy link
Copy Markdown
Contributor

It looks like something changed on 16.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-3069-by-TheMule71-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a9920a7 into OCA:16.0 Sep 22, 2023
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at c53d327. 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.