Code review comment for lp:~acysos-team/openerp-spain/openerp-spain-7.0-payment-order

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Ignacio, ya sé cuál es la razón por la que me sale el UnicodeError, y por qué a ti no te salía: porque rellenabas todos los datos correctamente y por eso no te aparecía ningún mensaje de error. La culpa es del sistema utilizado para mostrar los errores a la hora de la exportación, que es el mejor ejemplo de código "legacy". He hecho un pequeño hack para que funcione correctamente, aunque no me termina de convencer el sistema. De momento, como está pendiente el SEPA, voy a proceder a hacer el merge, y luego a ver si puedo perfilar algunas cuestiones filosas.

Muchas gracias por todo tu trabajo.

Un saludo.

review: Approve (code review and test)

« Back to merge proposal