Skip to content

[16.0] [MIG] l10n_it_rea#2988

Merged
OCA-git-bot merged 30 commits into
OCA:16.0from
TheMule71:16.0-mig-l10n_it_rea
Nov 10, 2022
Merged

[16.0] [MIG] l10n_it_rea#2988
OCA-git-bot merged 30 commits into
OCA:16.0from
TheMule71:16.0-mig-l10n_it_rea

Conversation

@TheMule71

Copy link
Copy Markdown
Contributor

No description provided.

@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_rea branch from c5a334d to 8c6c783 Compare November 2, 2022 02:14
@TheMule71 TheMule71 mentioned this pull request Nov 2, 2022
81 tasks
@primes2h

primes2h commented Nov 8, 2022

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_it_rea

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 8, 2022
@TheMule71 TheMule71 marked this pull request as ready for review November 10, 2022 09:59

@SirTakobi SirTakobi 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.

Grazie della PR!
Revisione del codice e provato in locale, per me è ok

@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_rea branch from 8c6c783 to ec48235 Compare November 10, 2022 13:50
hurrinico and others added 23 commits November 10, 2022 15:40
[FIX] Add readme

[FIX] fix province

[FIX] Module name and description

[FIX] Fixed Register

[FIX] rem descr from __openerp__
FIX bug tracker and empty keys
[MIG] Rename manifest files
[FIX] © -> Copyright, and make module installable

[FIX] Enconding on py files
[UPD] Readme
[FIX] openerp -> odoo in xml tag

Add Authors, use https on oca web site
ADD REA fields to copany and compute company_registry accordingly
Currently translated at 100.0% (15 of 15 strings)

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

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

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

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

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

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_rea
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_rea/it/
tafaRU and others added 7 commits November 10, 2022 15:40
…d and fix the following warnings that are getting runbot build red. (#1)

2020-11-20 10:30:20,958 160 WARNING openerp_test odoo.fields: res.company.rea_member_type: selection attribute will be ignored as the field is related
2020-11-20 10:30:20,959 160 WARNING openerp_test odoo.fields: res.company.rea_liquidation_state: selection attribute will be ignored as the field is related
Forward port of OCA#2257, fix for OCA#2256

Co-authored-by: SimoRubi <simone.rubino@agilebg.com>
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
Currently translated at 94.4% (17 of 18 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_rea
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_rea/it/
@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_rea branch from ec48235 to 9ae9fde Compare November 10, 2022 14:40

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

Both review: technical and functional 👍

@tafaRU

tafaRU commented Nov 10, 2022

Copy link
Copy Markdown
Member

/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-2988-by-tafaRU-bump-nobump, awaiting test results.

@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

@OCA-git-bot OCA-git-bot merged commit f7c0991 into OCA:16.0 Nov 10, 2022
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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