Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[13.0][MIG] l10n_es_toponyms #1200

Merged
merged 37 commits into from
Oct 15, 2019
Merged

[13.0][MIG] l10n_es_toponyms #1200

merged 37 commits into from
Oct 15, 2019

Conversation

dgomezquilon
Copy link
Contributor

Módulo testado y funcionando correctamente.

@oca-clabot
Copy link

Hey @dgomezquilon, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

  • Jordi Esteve jesteve@zikzakmedia.com (no github login found)
  • Borja L.S borjals@pexego.es (no github login found)
  • Launchpad Translations on behalf of openerp-spain-team (no github login found)
  • Omar (Pexego) (no github login found)
  • Omar (pexego) omar@pexego.es (no github login found)

Appreciation of efforts,
OCA CLAbot

@pedrobaeza pedrobaeza added this to the 13.0 milestone Oct 6, 2019
@pedrobaeza
Copy link
Member

Bueno, qué fin de semana más agitado has tenido! Si iba yo a hacer este PR. Tantas ganas de la 13.0 tienes?, jeje 😝

Fuera bromas, tienes que añadir en oca_dependencies.txt el repositorio partner-contact para "importar" las dependencias, y ya mezclé yo el de base_location_geonames_import para ello.

@pedrobaeza
Copy link
Member

Mañana revisaré lo otro que me comentas en otros PRs.

@OCA-git-bot OCA-git-bot mentioned this pull request Oct 6, 2019
Closed
35 tasks
@dgomezquilon
Copy link
Contributor Author

Bueno, qué fin de semana más agitado has tenido! Si iba yo a hacer este PR. Tantas ganas de la 13.0 tienes?, jeje 😝

Fuera bromas, tienes que añadir en oca_dependencies.txt el repositorio partner-contact para "importar" las dependencias, y ya mezclé yo el de base_location_geonames_import para ello.

Todos tenemos ganas de nuevas versiones 😝

No tenía claro cómo gestionáis el fichero oca_dependencies.txt y por eso no lo incluí. Ya lo he añadido y corregido el fallo de lint de travis.

Mañana revisaré lo otro que me comentas en otros PRs.

De acuerdo.

pedrobaeza
pedrobaeza requested changes Oct 11, 2019
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Para evitar este error (ya que el phone_validation se auto-instala dadas las dependencias):

2019-10-11 18:06:19,069 151 �[1;33m�[1;49mWARNING�[0m openerp_template odoo.addons.phone_validation.tools.phone_validation: The `phonenumbers` Python module is not installed, contact numbers will not be verified. Please install the `phonenumbers` Python module.

Añade phonenumbers a requirements.txt

l10n_es_toponyms/__manifest__.py Outdated Show resolved Hide resolved
@dgomezquilon
Copy link
Contributor Author

Para evitar este error (ya que el phone_validation se auto-instala dadas las dependencias):

2019-10-11 18:06:19,069 151 �[1;33m�[1;49mWARNING�[0m openerp_template odoo.addons.phone_validation.tools.phone_validation: The `phonenumbers` Python module is not installed, contact numbers will not be verified. Please install the `phonenumbers` Python module.

Añade phonenumbers a requirements.txt

@pedrobaeza ya está añadido.

@pedrobaeza
Copy link
Member

Un warning más (no se por qué no salió antes):

2019-10-12 12:28:25,052 161 WARNING openerp_test odoo.models: The model config.es.toponyms has no _description

Haz squash por favor de todos los commits de migración.

También estamos en pleno proceso en OCA de virar a un sistema de CI con pre-commit + autoformateador con black, pero bueno, si acaso hago la adaptación yo cuando lo tengamos estabilizado. Referencia: OCA/maintainer-quality-tools#613

@dgomezquilon
Copy link
Contributor Author

Un warning más (no se por qué no salió antes):

2019-10-12 12:28:25,052 161 WARNING openerp_test odoo.models: The model config.es.toponyms has no _description

Haz squash por favor de todos los commits de migración.

También estamos en pleno proceso en OCA de virar a un sistema de CI con pre-commit + autoformateador con black, pero bueno, si acaso hago la adaptación yo cuando lo tengamos estabilizado. Referencia: OCA/maintainer-quality-tools#613

Ya lo tienes @pedrobaeza

@emagdalenaC2i
Copy link

Code LGTM but travis is red

Borja L.S and others added 16 commits October 14, 2019 09:44
@pedrobaeza
[REF] *: Refactorización para portar a 6.0 - Paso 1.
cbc6d4c 
- 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

- Renombrados módulos para que los nombres estén en inglés.
@pedrobaeza
[FIX] l10n_es_toponyms: Corregido para ejecutarse correctamente a la …
a990d05 
…v 6.0 (TODO: retocar el asistente de configuración)
@pedrobaeza
[REF][REN][FIX] Corregimos errores en los nombres de dependencias de …
ba92d6e 
…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
@pedrobaeza
[FIX+IMP+REF] l10n_es_toponyms:
0679a37 
* FIX: Renombrado de los archivos de traducción catalanas de ca_ES.po a ca.po
* IMP: Adaptado asistente de configuración a la v 6.0
* IMP: Actualizadas traducciones ca y es
* IMP: Eliminadas reglas de seguridad en modelos osv.osv_memory
* REF: 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 openerp-spain@googlegroups.com
@pedrobaeza
[FIX] Use of ir module in l10n_es_toponyms_region and l10n_es_toponyms.
3f79c0d 
> [REF] l10n_es_toponyms: config.ES.toponyms -> config.es.toponyms.
> [ADD] l10n_es: Account journal sequence with references from l10n_es_pyme_account.
> [REF] l10n_es: Closed method for terceros_rec, terceros_pay and tax changed to unreconciled
@pedrobaeza
[FIX] l10n_es_toponyms _recover_zipcodes
b90ea49
@pedrobaeza
[FIX+IMP] l10n_es_toponyms: Varias cosas:
582e61a 
* FIX: Renombrado campo 'zipcode' a 'zip' en archivo l10n_es_toponyms_zipcodes.xml, del módulo 'l10n_es_toponyms', para que se pueda realizar la importación de ciudades.
* FIX: Añadida referencia del país (España) a cada ciudad que se importa cuando se tiene el módulo city.
* FIX: Error en evento onchange_city_info.
* FIX: Mapeado de la traducción a los nuevos nombres de campos (cambiados por el nuevo asistente).
* FIX: Recuperación de la información de ciudad corregida con los nuevos nombres de los campos de city.
* IMP: Introducida en el hilo principal de ejecución la importación de códigos postales.
* IMP: Pequeña limpieza de código.
@pedrobaeza
[MIG] l10n_es_toponyms: Migration to 7.0
c25a8c4 
Módulo de configuración de los topónimos españoles.

* Separado código para cuando se tiene el módulo city en un nuevo módulo llamado 'l10n_es_toponyms_city', dependiente de módulo 'city', para que sea más sencillo para nuevos usuarios.
* Corregida ruta para los archivos de las provincias.
* Añadida captura de pantalla.
* Códigos postales actualizados de GeoNames.
* Incluidas herramientas para extracción de los códigos postales.
* Descripción del módulo actualizada.
@pedrobaeza
[IMP] l10n_es_toponyms: Adaptado para utilizar como base el módulo 'b…
5e62157 
…ase_location'.
@pedrobaeza
Launchpad automatic translations update.
c387176
@pedrobaeza
[FIX] l10n_es_toponyms: Eliminación de códigos postales duplicados.
a587c6d
@pedrobaeza
[FIX] l10n_es_toponyms: 3 cosas:
7da208f 
* Refactorización PEP8 y convenciones v7
* Arreglado generador de códigos postales a partir de archivo de www.codigospostales.com.
* Quitado atributo type para quitar warning.
@sergio-teruel @pedrobaeza
[IMP] l10n_es_toponyms Nueva funcionalidad en gen_toponyms_geonames.p…
fa7c4da 
…y para añadir codigos INE en l10n_es_toponyms_zipcodes.xml
@acysos @pedrobaeza
[FIX] l10n_es_toponyms: fix vista version en vista formulario para Odoo
fee4fe7
@pedrobaeza
Launchpad automatic translations update.
58fd80f
@pedrobaeza
[MIG] l10n_es_toponyms: Modulo migrado a la v8
ff18257
etobella and others added 12 commits October 14, 2019 09:44
@etobella @pedrobaeza
[MIG] l10n_es_toponyms
389ddcd
@oca-transbot @pedrobaeza
OCA Transbot updated translations from Transifex
a41c40c
@mtbochaca @pedrobaeza
Translated using Weblate (català)
bd16d66 
Currently translated at 100,0% (16 of 16 strings)

Translation: l10n-spain-11.0/l10n-spain-11.0-l10n_es_toponyms
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-11-0/l10n-spain-11-0-l10n_es_toponyms/ca/
@sergio-teruel @pedrobaeza
[MIG] l10n_es_toponyms: Migration to 12.0
94df92d
@pedrobaeza
[MIG+FIX] l10n_es_toponyms: Adapt tests+code to 12.0
3dfcc5f
@oca-travis @pedrobaeza
[UPD] Update l10n_es_toponyms.pot
323dcf9
@mvrodriguez @pedrobaeza
Translated using Weblate (Galician)
bf1a322 
Currently translated at 100.0% (16 of 16 strings)

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_toponyms
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_toponyms/gl/
@oca-transbot @pedrobaeza
Update translation files
9ade3f4 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_toponyms
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_toponyms/
@josepmy @pedrobaeza
Translated using Weblate (Spanish)
b45c872 
Currently translated at 100.0% (15 of 15 strings)

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_toponyms
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_toponyms/es/
@josepmy @pedrobaeza
Translated using Weblate (Catalan)
ac2983e 
Currently translated at 100.0% (15 of 15 strings)

Translation: l10n-spain-12.0/l10n-spain-12.0-l10n_es_toponyms
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-l10n_es_toponyms/ca/
@OCA-git-bot @pedrobaeza
[UPD] README.rst
d2fa3c5
@dgomezquilon @pedrobaeza
[MIG] l10n_es_toponyms: Migration to 13.0
771d5a7
@pedrobaeza
Copy link
Member

@dgomezquilon no has habilitado el permiso a los maintainers para poder empujar a tu rama para solucionar el problema de black. ¿Me das acceso?

@dgomezquilon
Copy link
Contributor Author

@dgomezquilon no has habilitado el permiso a los maintainers para poder empujar a tu rama para solucionar el problema de black. ¿Me das acceso?

@pedrobaeza si me dices como te lo habilito.

@pedrobaeza
Copy link
Member

Mira a ver si tienes activa la opción de permitir a los maintainers ediciones:

Selección_016

He subido a https://github.com/Tecnativa/l10n-spain/tree/13.0-mig-l10n_es_toponyms lo que pretendía empujar aquí.

Lo que he hecho es:

@dgomezquilon
Copy link
Contributor Author

@pedrobaeza está activada.

@pedrobaeza
Copy link
Member

Uhm, entonces debéis tener algo más en vuestro fork que lo impide. Como te decía, tienes en esa rama los cambios, o los puedes replicar tú aquí para tener Travis limpio con lo nuevo de https://odoo-community.org/blog/the-oca-blog-1/post/black-isort-pre-commit-97

@dgomezquilon
Copy link
Contributor Author

@pedrobaeza ya están añadidos todos tus commits de vuestro fork en este PR.

@pedrobaeza
Copy link
Member

Buenas, hay algo raro, ya que el pre-commit sigue saliendo mal: https://travis-ci.org/OCA/l10n-spain/jobs/597992495, junto con un commit que yo no lo había puesto.

@dgomezquilon
Copy link
Contributor Author

Ahora si.

@pedrobaeza
Copy link
Member

/ocabot merge

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-1200-by-pedrobaeza-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 15, 2019
@OCA-git-bot
Merge PR #1200 into 13.0
a99141a 
Signed-off-by pedrobaeza
@OCA-git-bot OCA-git-bot merged commit 771d5a7 into OCA:13.0 Oct 15, 2019
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 08cd071. Thanks a lot for contributing to OCA. ❤️

@pedrobaeza
Copy link
Member

Fusionado! Gracias por la contribución.

pedrobaeza pushed a commit to sygel-technology/l10n-spain that referenced this pull request Oct 23, 2020
@OCA-git-bot
Merge PR OCA#1200 into 13.0
b53a1e8 
Signed-off-by pedrobaeza
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

@emagdalenaC2i emagdalenaC2i Awaiting requested review from emagdalenaC2i

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
13.0
Development

Successfully merging this pull request may close these issues.