Code review comment for lp:~vauxoo/openerp-venezuela-localization/ovl70-whsrc-rev-yani

Revision history for this message
hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha) wrote :

Si se instala el modulo sin data de demostrativa explota por que hace
referencia
a una vista que no existe en el modulo account: account_journal_view,

lo otro es que los datos que estan creando no estan bien organizados

hay referencias a xml_id que estan antes de la definicion de la xml_id

como esta referencia seq_account_retencion_src_customer, donde el
xml_id se referncia antes de la definicion

Este otro caso de abajo, es que aparece arriba definido
luego se referencia y por ultimo se vuelve a definir.

realmente son dos registros distintos que se han registrado con el
mismo xml_id, estos dentro de un mismo modulo deben ser unicos.

seq_type_account_wh_src_supplier

El orden de esa data xml habria de ser,

ir.sequence.type
ir.sequence
account.journal

2012/12/12 Humberto Arocha <email address hidden>

> Bajando branch para testear...
>
> les comento al finalizar.
>
> saludos
>
> 2012/12/11 Nhomar Hernandez (Vauxoo) <email address hidden>
>
> Review: Approve
>>
>> Por mi desde el punto de vista funcional hace todo lo esperado.
>> --
>>
>> https://code.launchpad.net/~vauxoo/openerp-venezuela-localization/ovl70-whsrc-rev-yani/+merge/139318
>> You are subscribed to branch
>> lp:~vauxoo/openerp-venezuela-localization/ovl70-whsrc-rev-yani.
>>
>
>

« Back to merge proposal