Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0] ADD l10n_it_withholding_tax_payment_order: Integrazione Ritenute d'acconto e Ordini di pagamento #3985

Merged

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented Feb 22, 2024

No description provided.

@eLBati eLBati force-pushed the 16.0-l10n_it_withholding_tax_payment_order branch from 1a225ed to 3060357 Compare February 22, 2024 15:14
@eLBati
Copy link
Member Author

eLBati commented Feb 22, 2024

Copy link
Contributor

@mrcast mrcast left a comment

Choose a reason for hiding this comment

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

Functional test: OK!

Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

LGTM

@eLBati eLBati force-pushed the 16.0-l10n_it_withholding_tax_payment_order branch from 3060357 to 9fa4dfa Compare May 24, 2024 07:52
@eLBati
Copy link
Member Author

eLBati commented May 24, 2024

Devo forse mettere i file .md in readme?

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 16.0.

@eLBati eLBati force-pushed the 16.0-l10n_it_withholding_tax_payment_order branch from 9fa4dfa to b0f9187 Compare May 24, 2024 14:54
@MaurizioConte
Copy link

@eLBati Lorenzo hai qualche novità su questa soluzione che stavi portando avanti ? Grazie

@SirAionTech SirAionTech added the needs fixing Has conflicts or is failing mandatory CI checks label Aug 21, 2024
@francesco-ooops
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 16.0.

@OCA-git-bot OCA-git-bot force-pushed the 16.0-l10n_it_withholding_tax_payment_order branch from b0f9187 to dd9cc62 Compare August 29, 2024 15:50
@francesco-ooops
Copy link
Contributor

@SirAionTech qualche idea sul perchè non si genera il readme?

@eLBati eLBati force-pushed the 16.0-l10n_it_withholding_tax_payment_order branch from dd9cc62 to a9f9312 Compare August 30, 2024 08:00
@eLBati
Copy link
Member Author

eLBati commented Aug 30, 2024

pre-commit verde

@SirAionTech SirAionTech added missing issue and removed needs fixing Has conflicts or is failing mandatory CI checks labels Aug 30, 2024
@francesco-ooops francesco-ooops linked an issue Aug 30, 2024 that may be closed by this pull request
12 tasks
@francesco-ooops
Copy link
Contributor

Relazionata a #4241

Copy link

@MaurizioConte MaurizioConte left a comment

Choose a reason for hiding this comment

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

TEST OK: Nell'ordine di pagamento viene riportato il valore della fattura al netto della ritenuta di acconto

@francesco-ooops
Copy link
Contributor

@OCA/local-italy-maintainers si può mergiare?

@andreampiovesana
Copy link
Contributor

merge?

@eLBati
Copy link
Member Author

eLBati commented Sep 3, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Sep 3, 2024
Signed-off-by eLBati
@OCA-git-bot
Copy link
Contributor

@eLBati your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-3985-by-eLBati-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@SirAionTech
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 16.0.

@OCA-git-bot OCA-git-bot force-pushed the 16.0-l10n_it_withholding_tax_payment_order branch from a9f9312 to 80ec8ad Compare September 4, 2024 07:15
@francesco-ooops
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Sep 4, 2024
Signed-off-by francesco-ooops
@OCA-git-bot
Copy link
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-3985-by-francesco-ooops-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9dab87d into OCA:16.0 Sep 4, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Ritenute: Rendicontazione Varie per i Fornitori
8 participants