Skip to content
New issue

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

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

Already on GitHub? # to your account

[16.0] Revert "[FIX] l10n_es_aeat_mod349: origin amount incorrect" #3729

Merged

Conversation

pedrobaeza
Copy link
Member

@pedrobaeza pedrobaeza commented Sep 19, 2024

Forward-port de #3727

This partially reverts commit 0e7847e, amended later by 0e93183.

Este parche fue metido sin advertirse en la migración a 14.0, cuando el PR original de 11.0 no estaba aprobado ni la solución estaba clara que fuera la adecuada:

imagen

y aparte de drenar rendimiento por asignar dos veces la variable origin_amount con mapeos que requieren obtener los datos, está provocando problemas de importes negativos. Luego más tarde en esta versión hubo una correción realizada en 0e93183, con su caso de uso, para la que se ha comprobado que una parte del código funciona, respetando esa parte.

@Tecnativa TT50922

This partially reverts commit 0e7847e,
amended later by 0e93183.
@pedrobaeza pedrobaeza added this to the 16.0 milestone Sep 19, 2024
@pedrobaeza
Copy link
Member Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3729-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4f43a20 into OCA:16.0 Sep 20, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants