Merge lp://staging/~mikel-martin/openerp-spain/6.1-ADD-evitar-asientos-duplicados-por-remesas into lp://staging/~openerp-spain-team/openerp-spain/6.1

Proposed by mikel
Status: Merged
Merged at revision: 354
Proposed branch: lp://staging/~mikel-martin/openerp-spain/6.1-ADD-evitar-asientos-duplicados-por-remesas
Merge into: lp://staging/~openerp-spain-team/openerp-spain/6.1
Diff against target: 438 lines (+131/-67)
2 files modified
l10n_es_payment_order/i18n/es.po (+123/-58)
l10n_es_payment_order/wizard/export_remesas.py (+8/-9)
To merge this branch: bzr merge lp://staging/~mikel-martin/openerp-spain/6.1-ADD-evitar-asientos-duplicados-por-remesas
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza Approve
Review via email: mp+173828@code.staging.launchpad.net

Description of the change

Se añade una comprobación, de forma que si se ha elegido "por pago
directo" como opción para crear los asientos contables, solo se pueda crear
el fichero de cobros/pagos si se han confirmado los pagos previamente.

Se evita así que se cree el asiento con los pagos y no quede la orden de cobro
en estado realizada.

Se añade también una comprobación para que si la orden de cobro está realizada
no se vuelvan a generar los asientos si se pulsa en "crear
el fichero de cobros/pagos"

To post a comment you must log in.
354. By mikel <mikel@hide>

[FIX] typo

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

Buenas Mikel.

Lo primero de todo, muchas gracias por la aportación. Siento haber tardado tanto en hacer una revisión, pero la tarea se me quedó traspapelada y no la he recuperado hasta ahora.

La funcionalidad que añades me parece perfecta. Sólo te pediría que antes de hacer el merge, corrijas por favor varias traducciones que no están bien del todo:

l. 197: Traducir "User error:\n"
l. 200: Traducir "Company %s has no vat."
l. 236: "El asistente no puede generar el archivo de remesas, no existen recibos." Poner dos puntos o un punto y seguido para la segunda frase.
l. 344: Cambiar "La empresa %s No tienes dirección de facturación." por "La empresa %s no tiene dirección de facturación."
l. 359: Cambiar la palabra "formailizarán" por "formalizarán" y poner una coma después del "Al crear el fichero"

Gracias.

Un saludo.

review: Needs Fixing
355. By mikel

[FIX] Correción textos traducción

Revision history for this message
mikel (mikel-martin) wrote :

Muchas gracias por la revisión, así da gusto contribuir ,)

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

¡Gracias por los cambios!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.