search

OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
265 stars 513 forks source link

[17.0][FIX]l10n_ca_es_cnae: correccion de autoria/eliminar .pot innecesarios #3652

Closed manuelregidor closed 4 weeks ago

manuelregidor commented 4 weeks ago

Antes de este PR había un error en la autoría del módulo.

También se han eliminado los ficheros .pot inncesarios.

T-6307

manuelregidor commented 4 weeks ago

@HaraldPanten @ValentinVinagre

HaraldPanten commented 4 weeks ago

/ocabot merge minor

FernandoRomera commented 4 weeks ago

@manuelregidor No se podria renombrar el módulo como "l10n_es_cnae_ca" y de esta manera si alguien busca sobre esta materia ambos módulos estaría contiguos. Ya que depende de "l10n_es_cnae". En la carpeta i18n, hay 3 ficheros .pot, creo que sobran 2.

OCA-git-bot commented 4 weeks ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 17.0-ocabot-merge-pr-3652-by-HaraldPanten-bump-minor, awaiting test results.

HaraldPanten commented 4 weeks ago

@manuelregidor

He eliminado la rama, a la espera de atender los comentarios sobre la carpeta i18n que ha hecho @FernandoRomera

En cuanto a renombrar el módulo, no está contemplado por nuestra parte, pero no estoy en contra si lo proponéis a futuro.

manuelregidor commented 4 weeks ago

@FernandoRomera @HaraldPanten He eliminado los ficheros .pot innecesarios y he puesto este cambio en un commit separado. He cambiado también el título del PR para que quede más claro. El cambio del nombre se puede proponer en el futuro.

HaraldPanten commented 4 weeks ago

/ocabot merge minor

OCA-git-bot commented 4 weeks ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 17.0-ocabot-merge-pr-3652-by-HaraldPanten-bump-minor, awaiting test results.

OCA-git-bot commented 4 weeks ago

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