-
-
Notifications
You must be signed in to change notification settings - Fork 520
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
[16.0][MIG] l10n_es_vat_book_oss: Migration to 16.0 #3157
Conversation
There was a problem hiding this 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
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.
There was a problem hiding this 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.
There was a problem hiding this 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.
Hola @aiendry-aktivsoftware, ¿podemos actualizar el PR? |
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. |
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. |
@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 👍 |
Debido a la falta de seguimiento, vamos a realizar nosotros el supersede y a seguir con el trabajo. |
Migration to version 16.0