Skip to content

[16.0][MIG] l10n_it_abicab#3036

Merged
OCA-git-bot merged 21 commits into
OCA:16.0from
tafaRU:16.0-mig-l10n_it_abicab
Nov 14, 2022
Merged

[16.0][MIG] l10n_it_abicab#3036
OCA-git-bot merged 21 commits into
OCA:16.0from
tafaRU:16.0-mig-l10n_it_abicab

Conversation

@tafaRU

@tafaRU tafaRU commented Nov 11, 2022

Copy link
Copy Markdown
Member

No description provided.

andrea4ever and others added 21 commits November 11, 2022 17:30
[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

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

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

@tafaRU

tafaRU commented Nov 14, 2022

Copy link
Copy Markdown
Member Author

Testata su runboat 👍

@tafaRU

tafaRU commented Nov 14, 2022

Copy link
Copy Markdown
Member Author

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit a8737e4 into OCA:16.0 Nov 14, 2022
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@tafaRU tafaRU deleted the 16.0-mig-l10n_it_abicab branch November 14, 2022 11:56
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.