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][MIG] l10n_es_vat_book_oss: Migration to 16.0 #3157

Conversation

aiendry-aktivsoftware
Copy link

@aiendry-aktivsoftware aiendry-aktivsoftware changed the title 16.0 mig l10n es vat book oss [16.0][MIG] l10n_es_vat_book_oss: Migration to 16.0 Jul 25, 2023
Copy link

@MayteGLC MayteGLC left a comment

Choose a reason for hiding this comment

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

LGTM

Se crea la factura INV/2023/00005 con posición fiscal Intra-EU B2C in Austria (EU-OSS-AT) a 08/01/2023

image

Al sacar el vat book se tienen en cuenta la base imponible de las facturas con posición fiscal Intra-EU B2C in Austria (EU-OSS-AT) base imponible de los impuestos OSS y las cuotas quedan a cero.

image

Copy link
Contributor

@ljsalvatierra-factorlibre ljsalvatierra-factorlibre left a comment

Choose a reason for hiding this comment

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

Please add some coverage and squash bots commits following the guidelines.


Por favor añade algo de cobertura de tests y haz squash de los commits siguiendo la guía de migración.

Copy link
Contributor

@ACheung-FactorLibre ACheung-FactorLibre left a comment

Choose a reason for hiding this comment

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

Hi @aiendry-aktivsoftware, please check the OCA guidelines for migrations. After squashing bots' commits, please improve the module's coverage.

@ljsalvatierra-factorlibre
Copy link
Contributor

Hola @aiendry-aktivsoftware, ¿podemos actualizar el PR?
¿Podrías revisar el PR @manuelregidor?

@HaraldPanten
Copy link
Contributor

HaraldPanten commented Dec 14, 2023

Hola, que las cuotas queden a 0 creo que no es incorrecto. Ahora no lo recuerdo de memoria, pero se habló sobre ello en su día aquí --> #1718 (comment)

¿En versiones anteriores aparece la cuota a 0, o aparece el importe?

Cualquier cosa me hacéis ping y estaré atento.

@HaraldPanten
Copy link
Contributor

En caso de que se retome este PR, tenéis que tener en cuenta esta corrección de la dependencia --> #3421 Gracias

@ljsalvatierra-factorlibre
Copy link
Contributor

En caso de que se retome este PR, tenéis que tener en cuenta esta corrección de la dependencia --> #3421 Gracias

Hola @aiendry-aktivsoftware, ¿tienes pensado realizar las modificaciones que te han solicitado? Gracias de antemano.

@HaraldPanten
Copy link
Contributor

@ljsalvatierra-factorlibre Creo que avanzará más si abrís vosotros PR y hacemos un supersede.

Pinguéame para cerrar este y seguir desde el vuestro cuando lo tengáis. Creo que avanzaremos mucho mejor así, ya que este PR ha quedado abandonado.

Gracias por el seguimiento 👍

@ValentinVinagre
Copy link
Contributor

Debido a la falta de seguimiento, vamos a realizar nosotros el supersede y a seguir con el trabajo.

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.