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.

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

[MIG] l10n_es_toponyms: Migration to 9.0 #404

Merged
merged 1 commit into from
Dec 7, 2016

Conversation

pedrobaeza
Copy link
Member

  • Update static l10n_es_toponyms_zipcodes.xml
  • Test in Python + improve speed

* Update static l10n_es_toponyms_zipcodes.xml
* Test in Python + improve speed
@pedrobaeza
Copy link
Member Author

cc @Tecnativa

@pedrobaeza pedrobaeza mentioned this pull request Nov 30, 2016
31 tasks
Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

Thanks, tested

@gonxi
Copy link

gonxi commented Dec 3, 2016

runbot is down with error 504.
I have tried to update the module and runbot has been down.
how rebuil or restart?
In this repositori is not better speak in spanish?
thanks

@pedrobaeza
Copy link
Member Author

Sí, puedes hablar en español, jeje. Es deformación profesional que se nos escapa a los que estamos todo el día en GitHub con el resto de repositorios, que hablamos en inglés. Pero vamos, como aquí todos somos políglotas, nos entendemos 😉

Está habiendo un error con runbot debido a la nueva infraestructura. Aquí se está debatiendo el error: OCA/runbot-addons#111.

Puedes probar a reconstruir a ver si está ya arreglado, pero creo que no.

@gonxi
Copy link

gonxi commented Dec 3, 2016

OK! Gracias por la info! Voy a probar a reconstruir y probar!

@pedrobaeza
Copy link
Member Author

Runbot arreglado!

@joserprieto
Copy link

joserprieto commented Dec 5, 2016 via email

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

Tested

@pedrobaeza
Copy link
Member Author

@joserprieto, es que se está utilizando, así que no hay posible debate...

@joserprieto
Copy link

@pedrobaeza, ya lo puse donde debe estar, si ves oportuno, borra mi comentario (o lo borro yo, como veas), para no "ensuciar" el hilo :)

@gonxi
Copy link

gonxi commented Dec 5, 2016

Ok! ya funciona runbot! para probar estos módulos se suele hacer alguna batería de test? es decir hara falta instalar minimo el de account para asignarle a un partner,no? se prueba tanto el fichero que lleva el modulo con los datos como la importación desde geonames?

@pedrobaeza
Copy link
Member Author

Esto no requiere el plan contable. Se puede ejecutar el asistente e instalar las provincias. Se han incluido de todas formas tests automáticos de prueba, que ya asegura el que esté en verde que mal del todo no va.

No sé si por timeout te va a dejar instalar todos los códigos postales, pero si quieres probar...

@gonxi
Copy link

gonxi commented Dec 5, 2016

ok! Lo decia porque al instalarlo si que se quedó frito por el timeout! pero en un principio el asisitente se lanza asi que supongo que "tested"! ;)

@antespi
Copy link
Contributor

antespi commented Dec 7, 2016

👍

@pedrobaeza pedrobaeza merged commit 86fcbe9 into OCA:9.0 Dec 7, 2016
@pedrobaeza pedrobaeza deleted the 9.0-l10n_es_toponyms branch December 7, 2016 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants