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_es_aeat_mod303: Error generación asiento contable cuando sale a pagar pero se compensa parcialmente con ejercicios anteriores #3874

Merged

Conversation

EmilioPascual
Copy link
Contributor

Fix para issue #3861

@ArantxaSudon @loida-vm @pedrobaeza revisad por favor.

@moduon MT-8455

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

"tax_ids": [
(
4,
tax_sale.id,
Copy link
Member

Choose a reason for hiding this comment

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

Evita aquí la coma final para no gastar tantas líneas. Por otro lado, ¿puedes evitar pasar tantos valores a mano y utilizar los estándar? Una cosa que puedes hacer es modificar los diccionarios de taxes para así poner solo una línea.

Copy link
Contributor

@loida-vm loida-vm left a comment

Choose a reason for hiding this comment

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

LGTM! Después de haber hecho los cálculos con facturas de clientes y proveedores, lleva correctamente las cantidades compensar a la 470 y las cantidades a ingresar a la 475
Gracias @EmilioPascual

Copy link
Contributor

@ArantxaSudon ArantxaSudon left a comment

Choose a reason for hiding this comment

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

Revisado funcionalmente

Generados los 303 de los cuatro trimestres con una casuistica distinta en cada trimestre, a pagar, a devolver, a compensar y a compensar una parte y pagar otra y se generan bien los asientos en cada uno de los casos.

En el último caso lleva perfectamente a la c/470 lo que compenso y a la c/4750 lo que debo ingresar

Muchas gracias @EmilioPascual

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Revisar mis comentarios anteriores.

@ArantxaSudon
Copy link
Contributor

Revisar mis comentarios anteriores.

Yo reviso funcionalmente porque no controlo de código pero tus comentarios siempre se tienen en cuenta para mejorar.

Gracias

@EmilioPascual EmilioPascual force-pushed the 16.0-fix_generacion_asiento_compensación_parcial branch from cce928f to 58c0ead Compare January 3, 2025 09:16
@EmilioPascual
Copy link
Contributor Author

Revisar mis comentarios anteriores.

Hecho, aunque del uso de los diccionarios de impuestos no estoy muy seguro, para esta comprobación se necesitan generar facturas con distintas cantidades en distintos trimestres y es independiente del impuesto elegido.

…ale a pagar pero se compensa parcialmente con ejercicios anteriores

@moduon MT-8455
@EmilioPascual EmilioPascual force-pushed the 16.0-fix_generacion_asiento_compensación_parcial branch from 58c0ead to fbc60f5 Compare January 7, 2025 07:56
@EmilioPascual
Copy link
Contributor Author

EmilioPascual commented Jan 20, 2025

¿@pedrobaeza por favor puedes volver a revisarlo cuando tengas tiempo? Muchas gracias.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

¡Gracias!

¿Tenéis pensado llevar esto a la 17?

/ocabot merge patch

@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-3874-by-pedrobaeza-bump-patch, awaiting test results.

@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). 🤖

@OCA-git-bot OCA-git-bot merged commit 21ef807 into OCA:16.0 Jan 20, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@EmilioPascual
Copy link
Contributor Author

¿Tenéis pensado llevar esto a la 17?

Si, aquí está.

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.

5 participants