-
-
Notifications
You must be signed in to change notification settings - Fork 533
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
[17.0][MIG] l10n_es_aeat_mod115: Migration to 17.0 #3541
[17.0][MIG] l10n_es_aeat_mod115: Migration to 17.0 #3541
Conversation
6f1e944
to
f7ee84a
Compare
/ocabot migration l10n_es_aeat_mod115 |
Any update? |
No. However i haven't forgoted this PR. When i have more time i will update this and finish the migration. Maybe next week or in the spanish OCA days |
82c30b6
to
7db6729
Compare
…do + tratamiento residentes 216 y cálculo
* Eliminar periodo específico y corregir exportación * Cálculo de importes de impuestos por cadena en lugar de recordset * Gestionar grupos de compañías
* Evitar problemas en campos related no readonly. Fixes 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. * Mejor herencia de impuestos 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.
… para diferenciar resultado
Currently translated at 94.3% (33 of 35 strings) Translation: l10n-spain-11.0/l10n-spain-11.0-l10n_es_aeat_mod115 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-11-0/l10n-spain-11-0-l10n_es_aeat_mod115/ca/
* Standard procedure * README by fragments * Improve code a bit * Expand tests for negative result
This way, we don't initially see all the subconfigs for each model and we can concentrate on main export configurations. This is got through `active` field default behaviour. Archived config are still directly searchable through the subconfig selector in the configuration line. All models export configs have been adapted to this new approach.
Currently translated at 100.0% (76 of 76 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod115 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod115/es/
Currently translated at 100.0% (76 of 76 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_aeat_mod115 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_aeat_mod115/ca/
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.
Technical review. LGTM
This PR has the |
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.
Revisión funcional correcta. LGTM
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
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.
Revisión funcional correcta con validación de la exportación BOE sobre el entorno de pruebas de la AEAT.
16df264
to
985dc61
Compare
@Tisho99 revisa los tests 😄 |
985dc61
to
59f30b0
Compare
The checks are fixed on the following PR: |
9d4f58f
to
1a0cf26
Compare
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at ba9508e. Thanks a lot for contributing to OCA. ❤️ |
No description provided.