Skip to content

[19.0][MIG] l10n_it_abicab: Migration to 19.0#5067

Merged
OCA-git-bot merged 27 commits into
OCA:19.0from
Borruso:19.0-mig-l10n_it_abicab
Jun 11, 2026
Merged

[19.0][MIG] l10n_it_abicab: Migration to 19.0#5067
OCA-git-bot merged 27 commits into
OCA:19.0from
Borruso:19.0-mig-l10n_it_abicab

Conversation

@Borruso

@Borruso Borruso commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

andrea4ever and others added 26 commits January 27, 2026 09:02
[IMP] porting module to Odoo 8.0

[ADD] Readme

clear __openerp__.py

[FIX] Flake8 - W292 no newline at end of file

Replace Openerp -> Odoo

[FIX] Depends - Readme - Module name - Update transalation file
[MIG] Rename manifest files
[FIX] authors

[FIX] copyright

[FIX] README syntax

[FIX] README syntax

[FIX] contributors
Currently translated at 100.0% (5 of 5 strings)

Translation: l10n-italy-11.0/l10n-italy-11.0-l10n_it_abicab
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-11-0/l10n-italy-11-0-l10n_it_abicab/it/
[IMP] Set partner bank abi/cab related
Currently translated at 100.0% (6 of 6 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_abicab
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_abicab/it/
Currently translated at 100.0% (9 of 9 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_abicab
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_abicab/it/
pylint with optional checks..............................................Passed
- hook id: pylint
- duration: 61.46s

************* Module l10n_it_fatturapa
l10n_it_fatturapa/views/partner_view.xml:51: [W7940(dangerous-view-replace-wo-priority), ]  Dangerous use of "replace" from view with priority 0 < 99
************* Module l10n_it_abicab
l10n_it_abicab/views/abicab_view.xml:29: [W7940(dangerous-view-replace-wo-priority), ]  Dangerous use of "replace" from view with priority 0 < 99
************* Module l10n_it_intrastat_statement.models.intrastat_statement_purchase_section
l10n_it_intrastat_statement/models/intrastat_statement_purchase_section.py:58: [R7980(consider-merging-classes-inherited), IntrastatStatementPurchaseSection1] Consider merging classes inherited to "account.intrastat.statement.purchase.section" from intrastat_statement_purchase_section.py:212, intrastat_statement_purchase_section.py:357, intrastat_statement_purchase_section.py:428.
************* Module l10n_it_intrastat_statement.models.intrastat_statement_sale_section
l10n_it_intrastat_statement/models/intrastat_statement_sale_section.py:26: [R7980(consider-merging-classes-inherited), IntrastatStatementSaleSection1] Consider merging classes inherited to "account.intrastat.statement.sale.section" from intrastat_statement_sale_section.py:172, intrastat_statement_sale_section.py:300, intrastat_statement_sale_section.py:371.

pylint with mandatory checks.............................................Failed
- hook id: pylint
- exit code: 4

************* Module l10n_it_fatturapa
l10n_it_fatturapa/views/partner_view.xml:51: [W7940(dangerous-view-replace-wo-priority), ]  Dangerous use of "replace" from view with priority 0 < 99
************* Module l10n_it_abicab
l10n_it_abicab/views/abicab_view.xml:29: [W7940(dangerous-view-replace-wo-priority), ]  Dangerous use of "replace" from view with priority 0 < 99
Removed method because there is a "related" attribute in
fields bank_abi & bank_cab which makes the method unnecessary
Steps:

 - install l10n_it_abicab
 - install l10n_mx

"ABM Code" field label disappears
@tafaRU

tafaRU commented Feb 6, 2026

Copy link
Copy Markdown
Member

/ocabot migration l10n_it_abicab

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Feb 6, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 6, 2026
30 tasks

@MarcoCalcagni MarcoCalcagni 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
test funzionali

@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). 🤖

@mmircoli-nexapp

Copy link
Copy Markdown

per caso è possibile mergiare, avendo due approvazioni?

@mmircoli-nexapp

Copy link
Copy Markdown

si può mergiare questa?

@tafaRU

tafaRU commented May 29, 2026

Copy link
Copy Markdown
Member

Lancio /ocabot rebase per rendere nuovamente disponibile runboat.

@tafaRU

tafaRU commented May 29, 2026

Copy link
Copy Markdown
Member

/ocabot rebase

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, PR rebased to 19.0.

@OCA OCA deleted a comment from OCA-git-bot May 29, 2026
@tafaRU

tafaRU commented May 29, 2026

Copy link
Copy Markdown
Member

Hi @sbidoul, could you please guide me on how to rebuild or regenerate the runboat build?
Thank you in advance 🙏

@sbidoul

sbidoul commented May 29, 2026

Copy link
Copy Markdown
Member

Hi Alex. The ocabot rebase command apparently did nothing. So a manual rebase or git commit --amend of the last commit is necessary.

@Borruso Borruso force-pushed the 19.0-mig-l10n_it_abicab branch from 641d123 to 4264475 Compare May 29, 2026 15:38
@tafaRU

tafaRU commented May 29, 2026

Copy link
Copy Markdown
Member

@sbidoul just as I suspected, thanks for the confirmation!

@sbidoul

sbidoul commented May 29, 2026

Copy link
Copy Markdown
Member

I can trade the runboat password to trigger rebuilds during OCA days :)

@tafaRU tafaRU left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Review tecnica e test funzionale su runboat 👍

@tafaRU

tafaRU commented May 29, 2026

Copy link
Copy Markdown
Member

anzi @Borruso piccola nota 😉 in base a quanto riportato su https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-19.0#technical-method-to-migrate-a-module-from-180-to-190-branch
nel commit di migrazione andrebbe riportato solo il bump di versione mentre la parte restante delle modifiche dovrebbe finire nel commit ...pre-commit auto fixes.

@tafaRU

tafaRU commented Jun 11, 2026

Copy link
Copy Markdown
Member

anzi @Borruso piccola nota 😉 in base a quanto riportato su https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-19.0#technical-method-to-migrate-a-module-from-180-to-190-branch
nel commit di migrazione andrebbe riportato solo il bump di versione mentre la parte restante delle modifiche dovrebbe finire nel commit ...pre-commit auto fixes.

in locale non avevo skippato l'hook oca-gen-addon-readme: ora mi torna tutto 👍

@tafaRU

tafaRU commented Jun 11, 2026

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit dd7f4ab into OCA:19.0 Jun 11, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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