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

Add module l10n_es_professional_category. #193

Closed
wants to merge 1 commit into from
Closed

Add module l10n_es_professional_category. #193

wants to merge 1 commit into from

Conversation

yajo
Copy link
Member

@yajo yajo commented Aug 3, 2015

Para resolver #174, se necesita saber la categoría profesional de los contactos en res.partner. Este módulo añade ese campo.

@pedrobaeza
Copy link
Member

Esta información, para el 95% de las empresas que usan el ERP, le es útil a nivel de empleado, no a nivel de res.partner. Ya sé que en tu caso con lo de las formaciones no es así, pero claro, el módulo así tal cual no es adecuado para su generalización.

@yajo
Copy link
Member Author

yajo commented Aug 3, 2015

Ese argumento ya se ha hablado varias veces, y creo que está claro que nuestro objetivo es diferente al de otras empresas, y es por ese motivo que los módulos son diferentes también.

Si nuestra empresa se dedicara a otra cosa (como ese 95% restante), no necesitaríamos hacer esto, pero intentamos cubrir con estos módulos un uso que no todo el mundo necesita (formación ajena). Quizá si estos módulos acaban estando disponibles para todo el mundo, haya más empresas de formación que se animen a usar Odoo y ese 95% descienda a un 90 u 80%. Si estos módulos continúan sin existir, definitivamente esos usuarios tendrán que recurrir a otras soluciones, y los que os dedicáis a vender Odoo tendréis más problemas para venderlos a empresas de formación.

Por otro lado, pedirme que me ponga a hacer el módulo adaptado para ese 95% cuando yo estoy en el 5% restante, comprenderás que es tan absurdo como pedirte a ti que hagas ese 5% por mí si a ti no te sirve para nada. Creo que todos juntos podemos llegar a cubrir el 100% de usos para Odoo, de eso se trata el trabajo en comunidad.

Si quieres puedo renombrarlo a l10n_es_partner_professional_category, y quien lo necesite en hr.employee, puede dedicar sus propios recursos a crear otro módulo llamado l10n_es_employee_professional_category sin que una cosa interfiera en la otra. También puedo hacer lo mismo con el resto de módulos de este estilo.

@pedrobaeza
Copy link
Member

Ya, pero no tiene sentido poner en la localización algo tan específico...

@yajo
Copy link
Member Author

yajo commented Aug 3, 2015

No hay un repositorio de formación ni eventos, y los requerimientos para comunicar cursos a la fundación tripartita son 100% específicos para España. ¿Dónde lo pongo si no?

@pedrobaeza
Copy link
Member

En uno específico tuyo, aunque público puede ser una solución. De todas formas, escuchemos la opinión de otros a ver.

@acysos
Copy link
Member

acysos commented Aug 3, 2015

Hola,

La localización en principio siempre se ha destinado a cubrir especificaciones propias del país. Yajo solo he revisado por encima este módulo, pero mi pregunta es directa, son aplicables a otras empresas de formación-docencia fuera de España? O son formatos específicos de la fundación tripartita?

Si la primera pregunta es afirmativa, es mejor crear en OCA un proyecto especifico para empresas de formación-docencia o incluso puede añadirse a otro repositorio como partner-contact. Si la segunda pregunta es afirmativa, entonces este es su repositorio.

Podrías aclarar un poco más, a primeras veo que añade un nuevo tipo de contacto, que no solo podría ser útil para empresas de docencia, se me esta ocurriendo alguna otra a nivel de profesionales autónomos. Los autónomos son partner pero puede clasificarse según su categoría profesional, a parte de la categoría general.

Saludos

@yajo
Copy link
Member Author

yajo commented Aug 3, 2015

La estructura que estoy siguiendo es dividir todo el proyecto en varios micromódulos independientes que siguen la misma lógica que dices, y 2 macromódulos.

En este caso, la categoría profesional es un parámetro específico de España, por eso este micromódulo lo envío a este repositorio. Por eso verás que incluye el fichero data/l10n_es_professional_category.category.csv en el que se precargan todas las categorías oficiales.

La categoría profesional es un campo que se puede utilizar en otros contextos dentro de la legislación española, por eso lo separo en otro módulo diferente.

Hay otros micromódulos parecidos que los mando a partner-contact, como OCA/partner-contact#149 (donde alguien me pregunta algo parecido a lo tuyo). En este caso, no sé si hay otro país que necesite un campo similar, pero tampoco me parece algo que deba estar intrínsecamente vinculado a España. Puede que un hospital chino se beneficie de ese micromódulo sin importar para qué lo haya creado yo.

La lista completa de PR que tengo en marcha la puedes ir viendo en #174.

Hay que admitir que, como dice @pedrobaeza, la mayoría de empresas que no sean de formación se beneficiarían más de tener la categoría profesional en hr.employee, pero como también ha dicho, no se trata del 100% de los casos. Si nadie se anima a portar este módulo a hr.employee, tan solo con vincular un hr.employee a un res.partner también podría usarse en ese 95% restante de casos.

En cuanto a los macromódulos, en principio habrá 2, llamados training y l10n_es_training. El primero adapta el módulo event para trabajar con formaciones, y no encaja en la estructura actual de la OCA, ya que no hay ningún repositorio relacionado con eventos (aunque sé que @pedrobaeza está en ello). El segundo iría en l10n-spain, vinculado al primero, y añadiría las funcionalidades específicas que requiere la Fundación Tripartita. Por supuesto, no puedo hacer PR de los macromódulos hasta que todos los pequeños se hayan integrado, pero la ventaja de hacerlo así es que incluso el macromódulo será también pequeño y muy fácil de testear.

@yajo yajo mentioned this pull request Aug 4, 2015
@yajo
Copy link
Member Author

yajo commented Aug 4, 2015

Closing. See #174 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants