← Back to team overview

avanzosc team mailing list archive

Re: [Merge] lp:~avanzosc/openerp-spain/7.0 into lp:openerp-spain/7.0

 

Review: Needs Fixing

Buenas, Daniel:

Gracias por el MP. He instalado y probado los módulos y, aunque funcionan más o menos bien, hay algunas cosas que no cuesta mucho que se pueden mejorar:

Módulo l10n_es_bank_statement
-----------------------------

- La entrada de menú "Cuentas por defecto asociadas a los códigos C43", que se encuentra dentro de "Configuración financiera > C43 bank statements", debido a la nueva organización de los menús, tiene más sentido que esté dentro de "Varios > Extractos bancarios", junto con la otra entrada que existe. También si se reduce el nombre a "Cuentas asociadas a los códigos C43, cabrá dentro del espacio reservado para los menús.
- La acción que lanza el menú "Asistente de importación de conceptos de e..." debería ponerse como un asistente de configuración (heredar de res.config.installer), que se lance al instalar el módulo. Cambiar el nombre a "Importación de códigos C43" también ayudaría para su interpretación.
- La ventana de importación de los extractos bancarios queda algo extraña (la casilla de verificación en una línea y el texto de la casilla en otra) debido a la configuración de la vista. Supongo que tendrá que ver con los atributos cols y colspan.

Módulo l10n_es_payment_order:
-----------------------------
- El campo "Crear asientos contables" aparece encima del nombre de la orden de cobro y sin etiqueta.
- Encima del todo aparece un campo no editable que pone "A cobrar", que corresponde al campo type.
- El asistente para "Crear fichero de cobros/pagos" no funciona. Esto último sí puede llevar más tiempo en corregir, ya que a lo mejor hay que cambiar el funcionamiento del mismo.
-- 
https://code.launchpad.net/~avanzosc/openerp-spain/7.0/+merge/159814
Your team Avanzosc Developers is subscribed to branch lp:~avanzosc/openerp-spain/7.0.


References