OCA / l10n-peru

Peruvian Localization
GNU Affero General Public License v3.0
13 stars 12 forks source link

[ADD][12.0] Add Peruvian toponyms module. #18

Closed JDSalcedo closed 4 years ago

JDSalcedo commented 4 years ago

Este PR agrega el módulo de topónimos peruanos como base para cualquier localización peruana. La data en los archivos de carga han sido construidos a partir de la información publicada por el Instituto Nacional de Estadística e Informática(INEI) de Perú, en su última actualización del 2016 disponible en la siguiente dirección:

http://webinei.inei.gob.pe:8080/sisconcode/proyecto/index.htm?proyectoTitulo=UBIGEO&proyectoId=3

Un saludo.

JDSalcedo commented 4 years ago

Hola @pedrobaeza , según el log de travis, falta el archivo README.rst para el módulo, no tengo muy claro según el template que menciona travis se genera automáticamente usando oca-gen-addon-readme, es generado al aceptarse el PR o como debo generarlo? tengo que descargar maintainer-tools?

JDSalcedo commented 4 years ago

Se completó con el README para el módulo, @pedrobaeza algún mantenedor que se pueda mencionar para pasar el PR?

pedrobaeza commented 4 years ago

@JDSalcedo ya eché un vistazo general y te guié en los primeros pasos, pero a partir de ahora debería revisarlo más gente local de allí.

JDSalcedo commented 4 years ago

Te agradezco mucho el apoyo y tu tiempo @pedrobaeza, sería genial que más personas en Perú puedan unirse a la iniciativa de esta localización, voy a escribir en los foros peruanos de Odoo. Corrígeme si me equivoco porfavor, ¿Actualmente no hay mantenedores de código para el repositorio de Perú no? Y para este y futuros PRs ¿Qué se necesita para poder hacer pasarlo?. ¿Cuál es tu concejo de cómo podríamos organizarnos?

pedrobaeza commented 4 years ago

Puedes hacer un llamamiento en listas para que la gente lo revise, y luego solicitar a oca ser psc de la localización para tener permisos.

JDSalcedo commented 4 years ago

Genial, voy a empezar llamando a más personas para que puedan revisarlo, y ¿estas personas deben haber firmado el CLA de OCA? O cualquiera puede comentar?

pedrobaeza commented 4 years ago

Para revisar no es necesario el CLA. Es para contribuir código para lo que hace falta.

JDSalcedo commented 4 years ago

Ok, entendido.

jwcaceres commented 4 years ago

Buena iniciativa, es útil

JDSalcedo commented 4 years ago

Gracias @jwcaceres , es para todos, puedes apoyarnos con validaciones, funcionales, saludos.

leolandeo commented 4 years ago

Ayer por la noche realice la instalación del módulo de topónimos Perú y la carga de la data suministrada se realizó muy rápido y todo corresponde a la realidad , mi mayor preocupación era el ubigeo correspondiente a Lima la cual empieza con 15 y viendo que en la tienda de odoo hay uno que lo consigna erróneamente con 14 , fue lo primero que verifique, comprobando que está correcto el trabajo realizado, con suerte y colaboración de la comunidad esperamos la localización peruana completa, funcional y Libre, gracias por su trabajo.

chochichaga commented 4 years ago

He realizado la prueba de instalación del modulo de los topónimos de Perú en ambas versiones (importando la data de INEI y con la copia local). En ambos casos la instalación fue correcta. Este modulo es muy util ya que es la base de cualquier otra localización que se quiera realizar. Gracias por el aporte!

pedrobaeza commented 4 years ago

@leolandeo @chochichaga utilizad la aprobación explícita de GitHub en el PR para que conste:

Selección_020

pedrobaeza commented 4 years ago

/ocabot merge nobump

OCA-git-bot commented 4 years ago

On my way to merge this fine PR! Prepared branch 12.0-ocabot-merge-pr-18-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 4 years ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot commented 4 years ago

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