[16.0][MIG] l10n_es_aeat_mod296: Migration to 16.0 #3184
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/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod296 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod296/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-15.0/l10n-spain-15.0-l10n_es_aeat_mod296 Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-15-0/l10n-spain-15-0-l10n_es_aeat_mod296/
|
@pedrobaeza ¿está todo bien en este ¡Gracias! |
|
Bueno, formalmente, el mensaje del último commit no es el establecido en la guía de migración. Funcionalmente, alguien debería revisarlo. |
|
/ocabot migration l10n_es_aeat_mod296 |
|
Buenas tardes @pedrobaeza ¿Alguien más lo tiene que validar? ¡Gracias! |
|
Faltaría lo del mensaje del commit y algún PSC o contribuidor habitual. De primeras, sabiendo lo que han significado otras migraciones, diría que habría que ajustar el listado de impuestos para que ocupe todo el ancho. |
polqubiq
force-pushed
the
16.0-mig-l10n_es_aeat_mod296
branch
4 times, most recently
from
0a993c6
to
91e52ca
Compare
polqubiq
force-pushed
the
16.0-mig-l10n_es_aeat_mod296
branch
from
91e52ca
to
0625ea6
Compare
Buenas tardes, @pedrobaeza , acabo de solucionar lo del mensaje del commit y lo del listado de impuestos para que ocupe todo el ancho. Entiendo que te referías a esto: Ya está arreglado. Ojalá algún PSC o contribuidor habitual nos pueda ayudar a tirar este PR para adelante. Saludos, |
|
Sí, a eso del ancho me refería, pero eso me hace dudar de que la otra revisión haya sido exhaustiva, ya que algo que se ve a simple vista, no se ha detectado. |
Bueno, realmente sí que lo detectamos, pero iba a hacer un Para las próximas veces lo haré como lo he hecho aquí, haciendo los cambios directamente en la migración. Saludos, |
|
OK, vamos a darle salida entonces según experiencia con la migración con otros módulos y vuestra revisión: /ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 5ece82b. Thanks a lot for contributing to OCA. ❤️ |
No description provided.