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

[15.0][FIX] l10n_es_aeat_mod349: Importe original incorrecto en rectificaciones de otros periodos #3468

Conversation

JordiBForgeFlow
Copy link
Sponsor Member

Relacionado con el modelo 349.

En el PR se incluye un primer commit que contiene un test que reproduce el escenario que falla. Se crean dos facturas para mismo proveedor en un periodo, y en el siguiente periodo se hacen abonos. En el segundo periodo del 349 se observa que el importe original en rectificaciones de otros periodos es incorrecto. Está duplicado.

Este primer commit falla el test.

En el segundo commit del PR se incluye la corrección. Este segundo commit ya no falla los tests.

When we enter refunds in another period, the rectifications section must correctly report the right amounts.
@OCA-git-bot
Copy link
Contributor

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

…ones de otros periodos

Se crean dos facturas para mismo proveedor en un periodo, y en el siguiente periodo se hacen abonos.
En el segundo periodo del 349 se observa que el importe original en rectificaciones de otros periodos es incorrecto. Está duplicado.
Copy link

github-actions bot commented Jul 7, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 7, 2024
@pedrobaeza pedrobaeza added this to the 15.0 milestone Jul 8, 2024
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 14, 2024
@pedrobaeza
Copy link
Member

@JordiBForgeFlow creo que el problema provenía del código incorrecto que se introdujo en la migración de 14.0 (sin ser parte de la migración), y que se está ya revirtiendo en #3727. Prueba cuando esté fusionado, y si sigues teniendo problemas, lo comentas para verlo. Por el momento, cierro aquí.

@pedrobaeza pedrobaeza closed this Sep 19, 2024
@pedrobaeza pedrobaeza deleted the 15.0-fix-l10n_es_aeat_mod349-fix-duplicated-original-refund-amt branch September 19, 2024 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants