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] FIX l10n_it_vat_settlement_date: avoid to copy l10n_it_vat_settlement_date #4474

Merged

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented Dec 2, 2024

For instance, bad case, when creating out refunds, settlement date could be set to previous month, causing wrong amounts in VAT statment

v14: #4473

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.

Test funzionale: OK

Copy link

@gdemarco2esseti gdemarco2esseti left a comment

Choose a reason for hiding this comment

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

Esito test funzionale: ok

@francesco-ooops francesco-ooops added the is porting This pull request is porting a change from another version label Dec 9, 2024
@francesco-ooops
Copy link
Contributor

/ocabot rebase

…_date

For instance, bad case, when creating out refunds, settlement date could be set to previous month, causing wrong amounts in VAT statment
@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 16.0.

@OCA-git-bot OCA-git-bot force-pushed the 16.0-fix-refund-l10n_it_vat_settlement_date branch from c8919af to 319401c Compare December 9, 2024 10:56
@francesco-ooops
Copy link
Contributor

@tafaRU porting buono per il merge?

@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 Dec 20, 2024

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 401a4e8 into OCA:16.0 Dec 20, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved is porting This pull request is porting a change from another version merged 🎉 missing issue ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants