[16.0][IMP] l10n_es_aeat_mod123: Add 2024 version #3528
Conversation
EmilioPascual
force-pushed
the
16.0-l10n_es_aeat_mod123-version_2024
branch
2 times, most recently
from
df68271 to
3917c74
Compare
|
He corregido el nombre de las casillas en el mapeo y la longitud de un parámetro en la configuración de exportación que estaba erronea. |
EmilioPascual
force-pushed
the
16.0-l10n_es_aeat_mod123-version_2024
branch
from
3917c74 to
0519cfe
Compare
There was a problem hiding this comment.
Se ha presentado el modelo 123 ya con los cambios realizados en este PR, y ha salido todo correcto!
Funcionalmente aprobado! LGTM!
Gracias @EmilioPascual
There was a problem hiding this comment.
Modelo 123 presentado correctamente con los cambios realizados en este PR.
Aprobado funcionalmente
@EmilioPascual muchas gracias
Cierto, tendría que ser una exportación a partir del 01/01/2024. Esperamos a los cambios. |
EmilioPascual
force-pushed
the
16.0-l10n_es_aeat_mod123-version_2024
branch
from
0519cfe to
2ad13b9
Compare
Efectivamente. La verdad que no tenía mucha idea de cómo había que hacerlo. Con los nuevos cambios que he subido creo que ya queda todo bien. Revisadlo de nuevo por favor. Gracias. |
| export_config = cls.env.ref( | ||
| "l10n_es_aeat_mod123.aeat_mod123_main_export_config" | ||
| def test_model_123(self): | ||
| self._invoice_purchase_create("2017-01-01") |
There was a problem hiding this comment.
Esto habrá que poner las fechas de 2024 para poder testear lo nuevo.
There was a problem hiding this comment.
Añadí un test para las fechas de 2024, de todas formas para que no haya equivocación he cambiado el nombre al test.
EmilioPascual
force-pushed
the
16.0-l10n_es_aeat_mod123-version_2024
branch
from
2ad13b9 to
5f08842
Compare
|
@pedrobaeza Si te cuadra, fusionamos y nosotros lo pasamos a V17 |
Creo que eso ya está. ¿Sólo hay que ponerlas en las configuraciones main, no? ¿O también hay que ponerlas en las páginas? |
There was a problem hiding this comment.
Sí, correcto, solo en la principal, que ya veo que está.
/ocabot merge major
@HaraldPanten de cara a su migración a v17, estaría bien convertirlo a CSV con el script que puse en otro PR de 17.
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 6c6b626. Thanks a lot for contributing to OCA. ❤️ |
Actualizado modelo 123 de AEAT para recoger los nuevos cambios para el 2024: https://www.agenciatributaria.es/static_files/AEAT_Desarrolladores/EEDD/Modelo_123/DR123e24.xls
Issues: #3520, #3227
Para el cálculo de impuestos del resto de rentas esta pendiente la inclusión de un nuevo impuesto en Odoo en el PR: odoo/odoo#161633. Posteriormente habrá que incluirlo en el mapeo de impuesto y en los test.
@pedrobaeza @HaraldPanten @ArantxaSudon @loida-vm @rafaelbn
MT-5770 @moduon