-
-
Notifications
You must be signed in to change notification settings - Fork 533
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_cnae: Migration to 17.0 #3616
Conversation
[ADD][l10n_es_cnae] CNAE 2009 codes Add categories for CNAE 2009 codes (those not found as NACE codes).
Currently translated at 100.0% (25 of 25 strings) Translation: l10n-spain-9.0/l10n-spain-9.0-l10n_es_cnae Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-9-0/l10n-spain-9-0-l10n_es_cnae/ca/
7425b54
to
7734046
Compare
Este módulo no hace falta así en esta forma, ya que está el general l10n_eu_nace en OCA/community-data-files |
@pedrobaeza pero faltan códigos que si están en el cnae. @HaraldPanten puedes confirmar? |
@pedrobaeza Sí es necesario, ya que hay códigos CNAE que no trae el módulo l10n_eu_nace, que son los definidos en este módulo. Sin embargo, sí son CNAE existentes : 10.21, 10.22... Puedes verlo aquí --> https://www.cnae.com.es/ Es por eso que creasteis este módulo ya hace años --> #414 (comment) |
ocabot migration l10n_es_cnae |
/ocabot migration l10n_es_cnae |
7734046
to
b40ff7e
Compare
Añado más info: En este listado --> https://ec.europa.eu/competition/mergers/cases/index/nace_all.html No están incluidos. |
OK entonces. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probado en runboat. LGTM.
@pedrobaeza go?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge nobump
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 7b87289. Thanks a lot for contributing to OCA. ❤️ |
La importación de las industrias se realiza desde el asistente del módulo l10n_eu_nace al no poderse generar directamente desde un fichero csv en esta versión, ya que las industrias padre no disponen de un identificador externo.