[14.0][ADD] modules integración Mod322 con Prorata especial #136
Conversation
|
I needed to test them outside the flow, as it was colliding with another module from nuobit |
|
@eantones Creo que esto ya está acabado. Podrías revisarlo y subirlo a desarrollo para comprobar que todo va bien? Gracias! |
|
@etobella Entiendo que este módulo no es específico para Oxigensalud, debería estar en un repo genérico no? ¿Como lo ves? |
|
En que repo crees que debería estar? En la OCA no puede estar, ya que depende de módulos de nuobit 🤔 IMO, creo que este sería su lugar ideal |
|
@etobella yo creo que si el módulo es genérico y puede usarse para otras empresas, aquí no tiene sentido que esté. Debería estar en un repo genérico. La OCA está claro que no, o se pone en un repo tuyo general y publico o en el repo de NuoBiT junto con el resto. |
|
@eantones Yo no voy a mantener un repositorio público con estas características. Además, teniendo en cuenta que el trabajo se ha realizado directamente y solo para oxigen no veo que daño hace aquí (el caso prorata especial con 322 no creo que sea muy habitual). En cualquier caso, este repositorio es público y puede usarse sin problemas, además que al ser trabajo realizado para oxigen, es correcto que mantengan ellos los derechos del código. No me parecería correcto ceder el código a Nuobit sin la confirmación expresa de Oxigen. |
|
@etobella totalmente de acuerdo en el tema de los derechos de código, esto ya estaba considerado. El punto aquí es que si fuesen módulos de la OCA, naturalmente se habrían subido a la OCA y no al repo del cliente, del mismo modo que creaste el modulo del 322 ya directamente en la OCA, obviamente con el permiso del cliente. |
|
@eantones Mi filosofía es que si es estándar está en la OCA. En este caso no puede estar ya depende de otros módulos que no están en la OCA, por lo tanto, deben ir a repositorio de Cliente. Hacerlo de otra forma sería crear una capa de implantador que se podría cerrar en un momento dado. Por lo tanto, en beneficio del cliente, opino que todo el código que no sea OCA debe ir a su repositorio |
|
@etobella los módulos son estándares, esto no es opinable, el resto que opine el cliente y lo que él decida me parecerá bien. |
|
@eantones tal vez no elegi la mejor palabra. Son estandares, pero no estan bajo el paraguas de la OCA. En ese caso siempre opino que deben ir bajo los repos de cliente. Si quieres hacemos un repo de 322 pero bajo la organizacion de oxigen |
|
@etobella el punto es que no deben mezclarse módulos específicos de un cliente con módulos genéricos y si lo ponemos en el repo de dónde has hecho el PR van a mezclarse. He creado otro repo dónde vamos a meter allí todos los módulos que sean genéricos, de esta forma no los mezclamos. Por favor, haz el PR contra el repo https://github.com/oxigensalud/odoo-community-addons. He visto que hay un par de módulos que son genéricos en el repo actual, cuando tenga un rato los muevo al nuevo y así ya lo dejamos todo limpio y correctamente clasificado. |
|
Superseded by oxigensalud/odoo-community-addons#1 |
Aprovecho para añadir tests (WIP)