OpenERP Spanish Localization Project

Code review comment for lp:~acysos-team/openerp-spain/openerp-spain-6.1-city

  1. openerp-spain-6.1-city
  2. Merge into 6.1
Revision history for this message
Ignacio Ibeas (www.acysos.com) (ignacio-acysos) wrote :

Tal y como esta ahora, l10n_es_toponyms detecta si esta o no instalado city y se comporta de forma diferente en cada caso. Yo lo dejaría así, es más versátil dependiendo las necesidades de cada usuario. Si que es cierto que si se hace depender de city, se puede refactorizar l10n_es_toponyms y reducir su código.

Además si alguien decide instalar a posteriori el módulo city, con actualizar l10n_es_toponyms actualizará todos los partners.

No se que opinarán el resto.

Saludos

« Back to merge proposal