OCA / l10n-mexico

Mexican Localization.
GNU Affero General Public License v3.0
3 stars 19 forks source link

[ADD] l10n_mx_toponyms #2

Closed agb80 closed 4 years ago

agb80 commented 8 years ago

@pedrobaeza @jbeficent @max3903 here you have the module l10n_mx_toponyms, please reviewe it and indicate me what else is missing for add into the OCA umbrella

max3903 commented 8 years ago

@agb80 Please add the following image in static/description: https://github.com/OCA/maintainer-tools/blob/master/template/module/static/description/icon.png

max3903 commented 8 years ago

@agb80 I just added the missing Travis configuration. Please rebase.

agb80 commented 8 years ago

Rebase done, working on comments from @pedrobaeza

agb80 commented 8 years ago

@pedrobaeza how manage external imports on OCA modules¿? The error in Travis is:

UserError: (u'Unable to install module "l10n_mx_toponyms" because an external dependency is not met: No module named unidecode', None)
pedrobaeza commented 8 years ago

You have to add a "requirements.txt" in the root folder including the needed libraries.