Code review comment for lp:~pexego/openerp-spain/removes_out_of_scope_modules_61

Daniel Campos (Avanzosc) (dani-ds) wrote :

Hola,
Me parecen bien los cambios que propones Omar, de hecho que varios de
los módulos que comentas los subí por que son módulos que solemos
utilizar aquí pero al no tener una rama propia si no es en la propia del
desarrollador no tenía bien claro donde deberían ir.

Saludos!

El 07/01/13 12:04, Omar (Pexego) escribió:
> Omar (Pexego) has proposed merging lp:~pexego/openerp-spain/removes_out_of_scope_modules_61 into lp:openerp-spain.
>
> Requested reviews:
> OpenERP Spain Team (openerp-spain-team)
>
> For more details, see:
> https://code.launchpad.net/~pexego/openerp-spain/removes_out_of_scope_modules_61/+merge/142091
>
> Hola,
>
> He hecho este merge proposal para que se delibere sobre los módulos que expongo abajo, ya que no considero que estén en el sitio correcto:
>
> nan_account_bank_statement: Es un módulo genérico que le vale a toda la comunidad no sólo a nosotros y por esto, está en la rama de account-payment, manejada por el grupo de openerp-community-reviewer. https://code.launchpad.net/~account-payment-team/account-payment/6.1
>
> hr_attendance_project: Desde luego está totalmente fuera de lugar. Se podría crear una rama hr-attendance en el grupo https://launchpad.net/~hr-core-editors, también manejado por openerp-community-reviewer.
>
> account_payment_extension: Es el mismo caso que nan_account_bank_statement, aun peor, que es un módulo que desde la revisión 1 está allí. https://code.launchpad.net/~account-payment-team/account-payment/6.1
>
> nan_account_payment_report: Lo mismo que el primero y el tercero, el problema de este es que no lo tenemos en otro sitio, pero considero que sería mejor proponerlo en la rama de account-payment. Además, depende de jasper-reports, creo que estaría bien evitar dependencias de este tipo en la localización española.
>
> account_invoice_analytics: No probé este módulo, pero tiene buena pinta y también creo que sería interesante para el resto de la comunidad, yo lo propondría en la rama de account-analytic, también manejada por openerp-community-reviewer. https://code.launchpad.net/~account-core-editors/account-analytic/6.1
>
> account_balance_reporting: Es un módulo que usa mucha más gente que españoles, por lo tanto, no podemos llevarlo por separado aquí, podríamos quizás tener una variante de el con algo específico para la localización española. Habría que proponerlo en alguna de las ramas de openerp-community-reviewer. Quizás: account-financial-report, o quizás crear una account-reporting.
>
> No se que opináis, pero para mí es importante tener bien delimitadas las ramas de openerp e irnos hacía la metodología de openerp-community-reviewer, propuesta por c2c hace unos meses, que es por lo que acabará apostando la comunidad bajo mi punto de vista y en el único sitio que tendremos módulos "extra" centralizados y de calidad, sin encontrarnos con que no están portados, cambios no avisados etc..., todos los problemas que teníamos en las ramas de extra-addons y la dispersión de módulos que hay ahora en cada rama de cada empresa.
>
> Un saludo
>

--
*Daniel Campos*
Técnico programador OpenERP, Avanzosc, S.L. Office phone / Tfono
oficina: (+34) 943 02 69 02
www.avanzosc.es

« Back to merge proposal