Code review comment for lp:~vauxoo/addons-vauxoo/7.0-account_voucher_category-dev-hbto

Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

@Hbto,
Viendo el módulo luce excelente!

Solo me surgieron 2 dudas:
1) Porque el archivo csv de permisoa tiene 1,0,0,0 para el manager; es decir, solo se podrán crear, no modificar, no leer, no eliminar...?
2) Te parece si el campo en account.voucher lo ponemos readonly en los estados diferente a draft?

Otro tema, que te quería preguntar, es que si te parece buena idea que el campo aparezca en la vista de aml editable para poder modificar o agregar a polizas existente dicho campo directamente desde aml o am

Un favor, lo puedes levantar en runbot, para ver que marque verde y probarlo directamente en linea.

« Back to merge proposal