-
-
Notifications
You must be signed in to change notification settings - Fork 520
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
[15.0][MIG]l10n_es_account_asset #2224
[15.0][MIG]l10n_es_account_asset #2224
Conversation
…s prevision tesoreria y activos/amortizaciones.
…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True
…hecho para aprovechar el estándar al máximo. * Incluido nuevo método de depreciación por porcentaje fijo.
* PEP8 * Icono del módulo * Valor por defecto para ext_method_time
* Nos traemos unas corecciones aplicadas en la versión 7.0 que no estaban en la 8.0, necesarias para hacer correctamente amortizaciones mensuales sin prorrata.
…rsiones de fecha. Fixes OCA#208
…sual ya incuidas en 7.0 + arreglado problema en la vista para la fecha de inicio de cálculo
[IMP] l10n_es_account_asset: tests added.
…e + remove # depreciations compensation. Fixes OCA#146 + Delete empty last depreciation line
…category_id (OCA#671) Pasos para reproducir el error: * Crear un tipo de activo * Al crear un nuevo activo, los datos de la periodicidad configurados en el tipo de activo no se importan al nuevo activo. Corrección: El método a sobreescribir debe ser el "onchange_category_id_values" y no el "onchange_category_id" tal y como es actualmente.
…CA#804) Corregido _compute_board_undone_dotation_nb para casos de prorrateo
…ins object instead of ids
…sset_management As `account_asset` is now enterprise, we switch the dependency to `account_asset_management` OCA module. This requires a heavy refactoring on code and tests, but with some hooks and fixes provided to OCA module, it's still very light. Other changes: * README by fragments * Promote to Production/Stable * Set maintainer
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_account_asset/es/
In some cases, the amount in annual_percentage is not synchronized with the one in the period one due to a non hunted bug in v10/v11. As in v12 now the important field is the annual one, we must synchronize it properly.
…w computed writable Since OCA/account-financial-tools#952, the onchange method for profile no longer exists, so there's an error when changing such profile when this module is installed. This commit fixes this and switch to new computed writable style, adding also tests for the profile assignation for better regressions detection. Closes OCA#1455
…tage As it alters computations for other methods. TT25453
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_account_asset/
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_account_asset/es/
Currently translated at 100.0% (9 of 9 strings) Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_account_asset Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_account_asset/ca/
Nos asignamos este PR ya que estamos con la migración del account_asset_management. Les daremos salida a ambos. |
/ocabot migration l10n_es_account_asset |
daa7849
to
c436f3c
Compare
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 👍🏻
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at f3967ce. Thanks a lot for contributing to OCA. ❤️ |
Standard migration
Depends: OCA/account-financial-tools#1377