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

[FIX][l10n_it_withholding_tax] Fix amount residual reconciled for inv… #3781

Merged
merged 2 commits into from
May 9, 2024
Merged

[FIX][l10n_it_withholding_tax] Fix amount residual reconciled for inv… #3781

merged 2 commits into from
May 9, 2024

Conversation

matteoopenf
Copy link
Contributor

@matteoopenf matteoopenf commented Dec 13, 2023 •
edited
Loading

Superseed di #3778 e ho mantenuto l'autore del commit Risolve #3712

@matteoopenf matteoopenf force-pushed the 14.0-23147-Fix_amount_to_reconcile branch 3 times, most recently from 4e8d8aa to fe1d9f6 Compare December 13, 2023 11:52
@matteoopenf matteoopenf marked this pull request as ready for review December 13, 2023 11:52
@matteoopenf matteoopenf mentioned this pull request Dec 13, 2023
Open
3 tasks
@patrickt-oforce
Copy link
Contributor

@tafaRU la issue è #3712

@francesco-ooops
Copy link
Contributor

https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue @tafaRU @TheMule71

@francesco-ooops francesco-ooops linked an issue Dec 18, 2023 that may be closed by this pull request
[14.0][l10n_it_withholding_tax] riconciliazione non coerente in caso di pagamento piu' grande #3712
Open
3 tasks
@sergiocorato
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0-23147-Fix_amount_to_reconcile branch from fe1d9f6 to b149c8c Compare February 27, 2024 18:12
@sergiocorato
Copy link
Contributor

Ho visto che c'è anche un problema sul Registra pagamento in caso di utilizzo dell'opzione per registrare un abbuono/spesa (ignora il valore della ritenuta d'acconto, imputandola completamente all'abbuono/spesa), come pure sull'abbinamento di pagamenti multipli tramite l'aggiungi in Debiti in sospeso (genera una serie di valori aggiuntivi per ogni riga abbinata).

Riesci a caricare un test per verificare cosa risolvi con questa?

@matteoopenf
Copy link
Contributor Author

Ho visto che c'è anche un problema sul Registra pagamento in caso di utilizzo dell'opzione per registrare un abbuono/spesa (ignora il valore della ritenuta d'acconto, imputandola completamente all'abbuono/spesa), come pure sull'abbinamento di pagamenti multipli tramite l'aggiungi in Debiti in sospeso (genera una serie di valori aggiuntivi per ogni riga abbinata).

Riesci a caricare un test per verificare cosa risolvi con questa?

Ciao @sergiocorato con questa siamo andati a risolvere con @alessandrocamilli il fatto che se si viene pagati in eccesso la fattura si chiude e non rimane aperta, come ci accadeva prima di questa PR.

andreampiovesana
andreampiovesana approved these changes Feb 29, 2024
View reviewed changes
Copy link
Contributor

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

LGTM

@matteoopenf matteoopenf force-pushed the 14.0-23147-Fix_amount_to_reconcile branch from b149c8c to 688da72 Compare March 18, 2024 11:09
@matteoopenf
Copy link
Contributor Author

@sergiocorato abbiamo fatto un ulteriore fix, fammi sapere se ti aiuta nel problema che avevi tu

@matteoopenf
Copy link
Contributor Author

ne per registrare un abbuono/spesa (ignora il valore della ritenuta d'acconto, imputandola completamente all'abbuono/spesa), come pure sull'abbinamento di pagamenti multipli tramite l'aggiungi in Debiti in sospeso (genera una serie di valori aggiuntivi per ogni riga abbinata).

il fix dovrebbe risolvere la casistica che esponi

sergiocorato
sergiocorato reviewed May 9, 2024
View reviewed changes
l10n_it_withholding_tax/models/account.py
vals.update(
{
"amount": invoice.amount_net_pay,
"credit_amount_currency": invoice.amount_net_pay,
Copy link
Contributor

Choose a reason for hiding this comment

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

Non so se ci sia la casistica di una fattura emessa in valuta con ritenuta d'acconto, sarebbe da verificare se la valuta della fattura è uguale alla valuta aziendale e in caso contrario convertire no?

@sergiocorato
Copy link
Contributor

Questa PR corregge questa casistica, potrebbe anche essere mergiata senza la modifica sulla valuta, visto che mi pare poco probabile, se non hai tempo di svilupparla.
Resta aperto l'altro problema, di cui direi si può aprire una issue a parte.

@andreampiovesana
Copy link
Contributor

merge?

@sergiocorato
Copy link
Contributor

sergiocorato commented May 9, 2024 •
edited
Loading

merge?

No @andreampiovesana , sto aspettando la risposta di @matteoopenf

@matteoopenf
Copy link
Contributor Author

merge?

No @andreampiovesana , sto aspettando la risposta di @matteoopenf

Guarda questa modica la abbiamo in produzione da un cliente e sta andando. Per la parte della valuta apriamo una issue a parte così da sistemarla a parte.
Che ne dici?
Grazie mille

@sergiocorato
Copy link
Contributor

/ocabot merge patch
Grazie.

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-3781-by-sergiocorato-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f01490d into OCA:14.0 May 9, 2024
4 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

This was referenced Jun 26, 2024
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andreampiovesana andreampiovesana andreampiovesana approved these changes

@sergiocorato sergiocorato sergiocorato approved these changes

Assignees
No one assigned
Labels
14.0 approved merged 🎉
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[14.0][l10n_it_withholding_tax] riconciliazione non coerente in caso di pagamento piu' grande
7 participants
@matteoopenf @patrickt-oforce @francesco-ooops @sergiocorato @OCA-git-bot @andreampiovesana @tafaRU