[FIX] l10n_es_aeat_mod347, 340: Fixed terms for 'real estate' #196
Conversation
| compute="_get_totals", string="Real state records") | ||
| total_real_state_amount = fields.Float( | ||
| compute="_get_totals", string="Real State Amount") | ||
| total_real_estate_transmissions_amount = fields.Float( |
There was a problem hiding this comment.
Debes poner oldname= y el nombre de la antigua columna para una migración automática. Cambia también el número de versión del módulo.
|
Runbot da error por un external_id de |
|
Efectivamente, es por eso que comentas. Lo documenté aquí: OCA/runbot-addons#50 |
|
Haz por favor un rebase, @alejandrosantana |
| @@ -25,7 +25,7 @@ | |||
| ############################################################################## | |||
| { | |||
| 'name': "Modelo 347 AEAT", | |||
| 'version': "1.2", | |||
| 'version': "1.3", | |||
There was a problem hiding this comment.
Por favor, sigue el nuevo formato de version de la OCA: https://github.com/OCA/maintainer-tools/blob/master/CONTRIBUTING.md#version-numbers
En este caso sería
'version': "8.0.1.3.0",
|
Revisión de código: 👍, después del cambio de versión |
…ds, variables, classes and translations, and related PEP8 fixes.
|
@pedrobaeza y @antespi Como tenía varios conflictos, sobreescribí mi rama a partir de la de OCA actualizada, con todos los cambios anteriores y el cambio de nomenclatura de versión y un PEP8 que surgió de los cambios. |
|
Gracias por el cambio. Paso a mergear ya puesto que está revisado. |
…ion_sources [FIX] l10n_es_aeat_mod347, 340: Fixed terms for 'real estate'
|
Gracias @alejandrosantana |
in fields, variables, classes and translations.
Renombrado todo lo relativo a 'real estate' (en vez de 'real state'), añadiendo la 'e' inicial en inglés, ya que el término correcto es así para referirse a temas inmobiliarios.