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.

Sign up for GitHub

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

Jump to bottom

[16.0] [MIG] l10n_it_vat_registries_split_payment #3394

Merged
merged 14 commits into from
Oct 13, 2023
Merged

[16.0] [MIG] l10n_it_vat_registries_split_payment #3394

merged 14 commits into from
Oct 13, 2023

Conversation

Borruso
Copy link
Contributor

@Borruso Borruso commented Jun 19, 2023 •
edited
Loading

Migration l10n_it_vat_registries_split_payment from 14.0 to 16.0

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@tafaRU
Copy link
Member

tafaRU commented Jun 20, 2023

/ocabot migration l10n_it_vat_registries_split_payment

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jun 20, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 20, 2023
Open
79 tasks
@DinamicheAziendali DinamicheAziendali mentioned this pull request Jun 29, 2023
Open
12 tasks
@tafaRU
Copy link
Member

tafaRU commented Jul 6, 2023

@Borruso controlla i tests. Grazie.

@Borruso Borruso force-pushed the 16.0-mig-l10n_it_vat_registries_split_payment branch from c92f54b to 72bf60d Compare July 6, 2023 10:02
@Borruso
Copy link
Contributor Author

Borruso commented Jul 6, 2023

@Borruso controlla i tests. Grazie.

test ok

@SirAionTech
Copy link
Contributor

SirAionTech commented Oct 2, 2023 •
edited
Loading

#3387 è stata mergiata, puoi aggiornare l'ultimo commit e fare rebase

@Borruso Borruso force-pushed the 16.0-mig-l10n_it_vat_registries_split_payment branch from 72bf60d to 1feebe0 Compare October 3, 2023 06:14
SirAionTech
SirAionTech requested changes Oct 3, 2023
View reviewed changes
Copy link
Contributor

@SirAionTech SirAionTech left a comment

Choose a reason for hiding this comment

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

Grazie della PR!
Puoi modificare la storia dei commit come descritto in https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate?

eLBati and others added 14 commits October 5, 2023 11:34
@eLBati @Borruso
ADD l10n_it_vat_registries_split_payment to fix the following case: (O…
02368a5 
…CA#570)

- Configure Split Payment fiscal position and tax
 - Make a customer invoice with SP
 - Print VAT register

Split Payment VAT must not appear as deductible VAT
@Borruso
[MIG] l10n_it_vat_registries_split_payment: Migration to 11.0
b46767b
@Borruso
[MIG] l10n_it_vat_registries_split_payment: Migration to 12.0
88d3f04
@Borruso
[FIX] Cambio logica di computazione delle tasse SP.
73c7c8a
@primes2h @Borruso
Added translation using Weblate (Italian)
bef5d24
@Borruso
merge PR 1264 1717 1728 1737 1739
42e57b6
@SimoRubi @Borruso
[IMP] l10n_it_vat_registries_split_payment: black, isort, prettier
2764283
@SimoRubi @Borruso
[MIG] l10n_it_vat_registries_split_payment: Migration to 14.0
95e80e3
@eLBati @Borruso
IMP l10n_it_vat_registries_split_payment tests: complete tax configur…
a1bf628 
…ation
@primes2h @Borruso
Translated using Weblate (Italian)
1a937c9 
Currently translated at 75.0% (3 of 4 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_vat_registries_split_payment
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_vat_registries_split_payment/it/
@TheMule71 @Borruso
[FIX] l10n_it_vat_registries_split_payment: fix test tags
f708c7f
@francesco-ooops @Borruso
Translated using Weblate (Italian)
28e9948 
Currently translated at 100.0% (4 of 4 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_vat_registries_split_payment
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_vat_registries_split_payment/it/
@Borruso
[IMP] l10n_it_vat_registries_split_payment: pre-commit stuff
8553467
@Borruso
[MIG] l10n_it_vat_registries_split_payment: Migration to 16.0
8f536b3
@Borruso Borruso force-pushed the 16.0-mig-l10n_it_vat_registries_split_payment branch from 1feebe0 to 8f536b3 Compare October 5, 2023 09:34
@Borruso
Copy link
Contributor Author

Borruso commented Oct 8, 2023

Grazie della PR! Puoi modificare la storia dei commit come descritto in https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate?

fatto

@OCA-git-bot
Copy link
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). 🤖

@tafaRU
Copy link
Member

tafaRU commented Oct 13, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3394-by-tafaRU-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 055bc98 into OCA:16.0 Oct 13, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

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

@SirAionTech SirAionTech SirAionTech approved these changes

@TonyMasciI TonyMasciI TonyMasciI approved these changes

Assignees
No one assigned
Labels
16.0 approved merged 🎉 migration needs functional review ready to merge
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.