search

ProyectoAconcagua / l10n_ar_point_of_sale_v7

Módulo encargado de adaptar los comprobantes de facturación, NC y ND a las normativas de Argentina.
3 stars 2 forks source link

Check for amount_total #5

Closed lbaldi closed 10 years ago

lbaldi commented 10 years ago

Se chequea que el monto total de una invoice sea positivo. En caso contrario no se permite validar lanzando error.

sebastiken commented 10 years ago

@lbaldi está bien solucionado, lo estuvimos viendo con @kloss17 Habría que actualizar los tests para cubrir este caso, pero igualmente, este chequeo lo mergeamos y después vemos lo del test. Saludos y gracias por colaborar.

sebastiken commented 10 years ago

@lbaldi para los mensajes de commits deberías tener en cuenta el formato que se utiliza en el proyecto en general. Los títulos en castellano y las etiquetas [ADD] [REM] [MOD] [FIX] para cada cosa realice el commit. Además en el mismo mensaje de commit podrías haber puesto al final _fixes #4 _ para que el Issue se cierra directamente con el commit y quede todo relaciones. Son solamente detalles a tener en cuenta. Igualmente, prometo terminar de redactar y subir todo este tipo de reglas lo antes posible.