Merge lp://staging/~vauxoo/openerp-mexico-localization/l10n_facturae_22_sep_payment_method_dev_carlos into lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_22-mod-sep-dev-yzk

Proposed by Juan Carlos Hernandez
Status: Merged
Merged at revision: 160
Proposed branch: lp://staging/~vauxoo/openerp-mexico-localization/l10n_facturae_22_sep_payment_method_dev_carlos
Merge into: lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_22-mod-sep-dev-yzk
Diff against target: 134 lines (+47/-6)
8 files modified
l10n_mx_facturae_22_payment_method/__openerp__.py (+1/-0)
l10n_mx_facturae_22_payment_method/__terp__.py (+1/-0)
l10n_mx_facturae_22_payment_method/data/payment_method_data.xml (+39/-0)
l10n_mx_facturae_22_payment_method/pay_method.py (+2/-2)
l10n_mx_facturae_22_payment_method/pay_method_view.xml (+1/-1)
l10n_mx_facturae_22_payment_method/pay_method_view_v5.xml (+1/-1)
l10n_mx_facturae_22_regimen_fiscal/regimen_fiscal.xml (+1/-1)
l10n_mx_facturae_22_regimen_fiscal/regimen_fiscal_v5.xml (+1/-1)
To merge this branch: bzr merge lp://staging/~vauxoo/openerp-mexico-localization/l10n_facturae_22_sep_payment_method_dev_carlos
Reviewer Review Type Date Requested Status
Juan Carlos Hernandez Needs Resubmitting
Moisés López - http://www.vauxoo.com Needs Fixing
Isaac López Zúñiga (community) Needs Fixing
Review via email: mp+115251@code.staging.launchpad.net

Description of the change

se agrego data demo para los metodos de pago probabdo para version 6 y 5

To post a comment you must log in.
Revision history for this message
Isaac López Zúñiga (isaako34) wrote :

Carlos, en la descripción de "Efectivo" tiene un \n y espacios, es importante quitarlos por favor.
Una mejora que se debe hacer es que debido a que el módulo solo depene de base y no de account, es necesario cambiar el menú de account hacia el menú de clientes, debe estar abajo de direcciones de cliente., lo mismo aplica para el módulo de Régimen fiscal.

review: Needs Fixing
161. By Juan Carlos Hernandez

[IMP][l10n_mx_facturae_22_payment_method]quit \n

162. By Juan Carlos Hernandez

[IMP][regimen_fiscal,payment_method]Change menu and remove space in field efectivo

Revision history for this message
Juan Carlos Hernandez (openerp1) wrote :

Cambiar el menú de regimen fiscal hacia el siguiente menú Ventas/Libreta de direcciones,abajo de direcciones de cliente
en la descripción de "Efectivo" se quito un \n y espacios,
Cambiar el menú de metodo de pago hacia el siguiente menú Ventas/Libreta de direcciones,abajo de direcciones de cliente

review: Needs Resubmitting
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

Juan Carlos,

Está todo perfecto.

Un favor, el cambio del menú siempre va a ser en:
Sales/Configuration/Address Book/*

review: Needs Fixing
163. By Juan Carlos Hernandez

[IMP][metodo de pago]change location to Sales/Configuration/Address Book/*

164. By Juan Carlos Hernandez

[IMP][regimen fiscal]change location to Sales/Configuration/Address Book/*

Revision history for this message
Juan Carlos Hernandez (openerp1) wrote :

Listo se cambio metodo de pago y regimen fiscal a Sales/Configuration/Address Book/* en la version 6.1
Listo se cambio metodo de pago y regimen fiscal a Partners/Address Book/* en la version 5

review: Needs Resubmitting
165. By Juan Carlos Hernandez

[IMP][metodo de pago]change view to Partners/Address Book/ for the open version5

166. By Juan Carlos Hernandez

[IMP][regimen fiscal]change view to Partners/Address Book/ for the open version5

Revision history for this message
Isaac López Zúñiga (isaako34) wrote :

Listo Carlos, los menús se crearon correctamente al igual que la carga de la data. Gracias.

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.

Subscribers

People subscribed via source and target branches