[19.0][MIG] l10n_es_aeat_mod296: Migration to 19.0 - #4933
Conversation
* README, manifiesto e icono * Ajustes y mejoras * Cálculo por códigos de impuestos generalizado + tratamiento residentes 216 y cálculo * Traducciones * Reglas de acceso para las líneas del informe
…lugar de recordset + gestionar grupos de compañías
…ixes OCA#290 Cuando se tienen campos related, aunque el valor no se cambie, Odoo lanza una escritura en el objeto relacionado cada vez que se guarda el objeto principal. En el caso de la moneda asociada a la compañía, hace que aquellos usuarios que no tengan permiso para modificar la compañía (la mayoría), falle en la creación del modelo.
El anterior método utilizado, al tener una clase y un many2one genérico, no permitía poner los datos en múltiples modelos hijos, ya que se repetían los IDs, y cogía los datos de un modelo en otros. Con este nuevo sistema, se sigue utilizando una tabla, pero con doble índice: el modelo y el ID, parecido a un campo reference, pero gestionado por código propio en el método de cálculo. El 303 está adaptado a este nuevo sistema, y se incluye un script de migración que conserve los datos.
Currently translated at 100.0% (105 of 105 strings) Translation: l10n-spain-10.0/l10n-spain-10.0-l10n_es_aeat_mod296 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-10-0/l10n-spain-10-0-l10n_es_aeat_mod296/es/
Standar procedure.
Same change as done in model 216
Currently translated at 100.0% (146 of 146 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod296 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod296/es/
* Include chatter (mail.thread) * Include activities (mail.activity.mixin) * Track AEAT report state change * Include new admin user in AEAT group by default
…n tests This way, we can reuse it in other modules that don't inherit AEAT report like SII. We also clean a non needed reading of the tax description field. Tests were still using description method for populating data, so we switch to new system and perform needed changes in dependent tests. s
It was outdated. Fixes OCA#1380
[UPD] Update l10n_es_aeat_mod296.pot [UPD] README.rst [UPD] Update l10n_es_aeat_mod296.pot
TT37373 l10n_es_aeat_mod296 13.0.1.2.0 Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-13.0/l10n-spain-13.0-l10n_es_aeat_mod296 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-l10n_es_aeat_mod296/
Currently translated at 85.1% (143 of 168 strings) Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_aeat_mod296 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_aeat_mod296/es/
…a has been included in l10n_es module MT-7610
…agencia [IMP] l10n_es_aeat*: cambio de menu a declaraciones con submenus por agencia [IMP] l10n_es_aeat*: cambio de menu a declaraciones con submenus por agencia
|
/ocabot migration l10n_es_aeat_mod296 |
ArantxaSudon
left a comment
There was a problem hiding this comment.
Revisión funcional
Recoge perfectamente las retenciones y las bases de no residentes y sus perceptores
El único problema que he localizado es que el modelo 296 es un modelo anual y no me deja seleccionar esa opción. He hecho las comprobaciones por trimestres pero cuando he ido a comprobar todo el año solo he podido marcando un trimestre y cambiando manualmente las fecha pero deja marcado el trimestre que he seleccionado
Necesitamos añadir la opción ANUAL
Muchas gracias @EmilioPascual @moduon MT-14058
|
El módulo tenía un error y no estaba puesto como declaración anual. @ArantxaSudon revisa de nuevo por favor. |
The 296 report is annually. https://sede.agenciatributaria.gob.es/Sede/procedimientoini/GI22.shtml MT-14058 @moduon
4d9d692 to
3aeb999
Compare
ArantxaSudon
left a comment
There was a problem hiding this comment.
Revisado de nuevo y ahora si recoge todo perfectamente y el modelo es anual
Muchas gracias @EmilioPascual @moduon MT-14058
|
This PR looks fantastic, let's merge it! |
|
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 19.0-ocabot-merge-pr-4933-by-pedrobaeza-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
This PR has the |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at cc51c42. Thanks a lot for contributing to OCA. ❤️ |
Depends on:
MT-14058 @moduon