Merge lp://staging/~vauxoo/openerp-mexico-localization/7.0_changes_report_facturae_dev_luis into lp://staging/openerp-mexico-localization

Proposed by Luis Torres - http://www.vauxoo.com
Status: Superseded
Proposed branch: lp://staging/~vauxoo/openerp-mexico-localization/7.0_changes_report_facturae_dev_luis
Merge into: lp://staging/openerp-mexico-localization
Diff against target: 272 lines (+96/-87)
4 files modified
l10n_mx_facturae_report/data.xml (+6/-5)
l10n_mx_facturae_report/report/invoice_facturae_html.mako (+88/-82)
l10n_mx_res_partner_bank/res_bank.py (+1/-0)
l10n_mx_res_partner_bank/res_bank_view.xml (+1/-0)
To merge this branch: bzr merge lp://staging/~vauxoo/openerp-mexico-localization/7.0_changes_report_facturae_dev_luis
Reviewer Review Type Date Requested Status
Luis Torres - http://www.vauxoo.com (community) Needs Resubmitting
Moisés López - http://www.vauxoo.com Needs Fixing
Isaac López Zúñiga Pending
Review via email: mp+167883@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2013-06-07.

Description of the change

- Se cambio la posicion del logo de la compañia
- Se cambio la leyenda transversal
- se agrego el campo de referencia para cada uno de losbancos
- se agrego el nombre del banco en la parte de los ultimos cuatro digitos de la cuenta bancaria

To post a comment you must log in.
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

Luis,
La sentencia
o.acc_payment.bank_name te va a marcar error si viene vacía.
Debería de ser
o.acc_payment and o.acc_payment.bank_name.bank_name or ''

review: Needs Fixing
220. By Luis Torres - http://www.vauxoo.com

[FIX][l10n_mx_facturae_report]Added validation that invoice have acc_number

Revision history for this message
Luis Torres - http://www.vauxoo.com (luis-cleto-) wrote :

De echo ese error en webkit no lo marca, pero ya agregue esta validacion

review: Needs Resubmitting

Unmerged revisions

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.