Code review comment for lp:~angelmoya/openerp-spain/domatix_special_periods

Revision history for this message
Omar (Pexego) (omar7r) wrote :

Hola Ángel,

He revisado los cambios y probado a instalar una bbdd limpia con el pyme_account, el funcionamiento fue correcto, pero los periodo creados son los de OpenERP y los especiales de tu mejora, OpenERP crea los periodos con el nombre MES/AÑO y nosotros en la localización española seguimos otro formato, que es NUM_PERIODOTAÑO[2:] y para los especiales, tu los creas como A/AÑO por ejemplo y el formato original era AAÑO[2:], no se que le parece al resto, este cambio en principio no es significativo. A ver que dicen. Esperamos a su validación y se puede proceder con el merge.
Probé también luego a generar otro plan contable para otra copañía y también parece que funciona correctamente.
Hice una pequeña refactorización del código ya te subo un commit ahora. Perdona la tardanza.
Una cosa que falta es la traducción.

Saludos

« Back to merge proposal