[17.0][MIG] l10n_es_partner: Migration to 17.0 #3325
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Se han renombrado los módulos para usar la nomenclatura propuesta por OpenERP: l10n_es para el módulo base de localización (plan de cuentas), l10n_es_* para el resto de módulos. - Se eliminan los módulos extra_addons/* que deberían moverse a los extra-addons genéricos (no son específicos de España). - Se renombran los __terp__.py por __openerp__.py
…ciertos módulos, por los correspondientes que se modificaron para esta versión 6.0 y en ciertos __init__ adaptamos los imports a los nuevos nombres de los ficheros. Renombrado de los archivos de traducción españoles de es_ES.po a es.po y pequeñas refactorizaciones
[FIX] Renombrado de los archivos de traducción catalanas de ca_ES.po a ca.po [IMP] Eliminadas reglas de seguridad en modelos osv.osv_memory [IMP] Añadido website en la definición del módulo [IMP] Actualizado listado bancos y cajas españoles con los últimos datos disponibles del Banco de España: http://www.bde.es/webbde/es/secciones/servicio/regis/regent_excel.html [IMP] Improved the introduction of the VAT (uppercase conversion). [RFR] Cambio de licencia de GPL3 a AGPL3 en el resto de módulos de localización española, con el consentimiento de todos los autores expresado vía lista de correo del equipo de localización [email protected]
* [IMP] Cambio de la denomición del banco 0216. Actualizado por BDE el 22/11/2011 * [IMP] Añadido ING DIRECT a las lista de bancos * [FIX] Añadido campo code, eliminado en res.bank de la versión 6.1. * [FIX] Bug #950256
…ercial name field.
…emoved from res.bank so I have added it in the localization as it is required to lookup banks when entering the bank account number. It isn't a normal merge cause of different bzr version in the branches
…on los últimos datos disponibles del Banco de España: http://www.bde.es/webbde/es/secciones/servicio/regis/regent_excel.html
…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
[IMP] Refactorización de código [IMP] Asistente de importación puesto como res.config.installer que se ejecuta al instalar el módulo. [IMP] Limpieza del archivo XML de bancos. [FIX] Adaptación de las vistas a la versión 7.0. [IMP] PEP8 + mejoras en la herencia de vistas Launchpad automatic translations update.
…ltima versión Launchpad automatic translations update.
[IMP] l10n_es_partner: Mejorada descripción del módulo. [IMP] l10n_es_partner: Incluido icono. [IMP] l10n_es_partner: Añadido BIC de Pastor Serfin. [FIX] l10n_es_partner: Eliminados espacios antes de comprobar CCC para que dé correcto. [IMP] l10n_es_partner: Eliminación de la mayoría de warnings que da runbot. Launchpad automatic translations update.
* [FIX] Cerrar manejador de fichero y clase en PEP8 * [FIX] Campo code duplicado * [IMP] Test * [FIX] Añadida dependecia a base_vat * [FIX] Aplicar API 8.0 y mas test * Importar los toponimos
…t de generacion de data_banks.xml
…y + Eliminar apóstrofes de los identificadores externos + Forzar cambio de id externo para el banco res_bank_bnpparibasse
Se ha corregido además el script de importación para que ya no cause más problemas OCA Transbot updated translations from Transifex
[ADD] Mas caracteres a escapar
…ved on update As the data generated by `l10n.es.partner.import.wizard` is created using the flag `noupdate=False`, the subsequent update of the module is removing all the created data. This causes the Bank Accounts (`res.partner.bank`) to be silently losing their assigned Bank (res.bank). OCA Transbot updated translations from Transifex
* Better search on partner * Review fixes * More tests * Migration script * Improve search over other terms: When searching on the omnibar, you can include other filters and the default search includes other different arguments, so the condition for adding the commercial name should be expanded. * Make name_search work As Odoo overwrites name_search on res.partner in a non inheritable way, we have to include explicitly name_search method and search first for the partners with that commercial, and then the rest. OCA Transbot updated translations from Transifex
OCA Transbot updated translations from Transifex
* Updated banks local file. * Updated import script * Adapted res.partner.bank logic and views * Adapted account.journal logic
OCA Transbot updated translations from Transifex
javierobcn
force-pushed
the
17.0-mig-l10n_es_partner
branch
from
abca26d to
0f31712
Compare
|
@javierobcn Puedes hacer rebase para recrear runboat? gracias. |
|
Los tests no parecen estar correctos. |
javierobcn
force-pushed
the
17.0-mig-l10n_es_partner
branch
from
df310bd to
b4c63da
Compare
Hecho! @HaraldPanten Puedes revisar |
miquelalzanillas
approved these changes
Feb 14, 2024
Ya pasan, dale un vistazo cuando puedas. Gracias |
HaraldPanten
approved these changes
Feb 14, 2024
|
@pedrobaeza Si consideras que se han tenido en cuenta los comentarios descritos en la revisión técnica, adelante. |
pedrobaeza
reviewed
Feb 14, 2024
javierobcn
force-pushed
the
17.0-mig-l10n_es_partner
branch
3 times, most recently
from
0276f90 to
ed2c3e5
Compare
pedrobaeza
reviewed
Feb 16, 2024
javierobcn
force-pushed
the
17.0-mig-l10n_es_partner
branch
from
ed2c3e5 to
d2212d2
Compare
|
@pedrobaeza los comentarios fueron atendidos. Si lo ves OK, creo que podemos irlo fusionando. Gracias. |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at c3451d9. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Work in progress