Skip to content

[16.0] [MIG] l10n_it_fiscal_payment_term#2992

Merged
OCA-git-bot merged 16 commits into
OCA:16.0from
TheMule71:16.0-mig-l10n_it_fiscal_payment_term
Nov 11, 2022
Merged

[16.0] [MIG] l10n_it_fiscal_payment_term#2992
OCA-git-bot merged 16 commits into
OCA:16.0from
TheMule71:16.0-mig-l10n_it_fiscal_payment_term

Conversation

@TheMule71

@TheMule71 TheMule71 commented Nov 2, 2022

Copy link
Copy Markdown
Contributor

Incorpora #2990 per i test

@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_fiscal_payment_term branch from cca0cbc to 72dfbde Compare November 2, 2022 02:51
@TheMule71 TheMule71 mentioned this pull request Nov 2, 2022
81 tasks
@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_fiscal_payment_term branch from 72dfbde to 2075af5 Compare November 2, 2022 19:40
@primes2h

primes2h commented Nov 8, 2022

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_it_fiscal_payment_term

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 8, 2022
@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_fiscal_payment_term branch from 2075af5 to bd25995 Compare November 10, 2022 10:43
@TheMule71 TheMule71 marked this pull request as ready for review November 10, 2022 10:44
@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_fiscal_payment_term branch 3 times, most recently from 55ba95d to 383ff87 Compare November 10, 2022 13:55

@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!
Ho fatto revisione del codice e l'ho provato in locale, per me è ok

sergiocorato and others added 15 commits November 11, 2022 17:15
[FIX] removed untranslated po and change fields name for more general purpose

[FIX] ref to fiscal payment in xml demo

[IMP] add migration script
and openerp XML tag
Currently translated at 100.0% (12 of 12 strings)

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

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fiscal_payment_term
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fiscal_payment_term/it/
…pa.payment_method or fatturapa.payment_term

Every code is already added by the module and user could ignore the right data to insert
@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_fiscal_payment_term branch from 383ff87 to 683c5db Compare November 11, 2022 16:15
@TheMule71 TheMule71 force-pushed the 16.0-mig-l10n_it_fiscal_payment_term branch from 683c5db to 26c014b Compare November 11, 2022 16:29
@eLBati

eLBati commented Nov 11, 2022

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-2992-by-eLBati-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 11, 2022
Signed-off-by eLBati
@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-2992-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 31f418a into OCA:16.0 Nov 11, 2022
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

9 participants