Code review comment for lp:~danimaribeiro/openerp.pt-br-localiz/openerp.pt-br-localiz

Revision history for this message
Renato Lima - http://www.akretion.com (renatonlima) wrote :

Olá Danimar,

Você poderia fazer um revert apenas do l10n_br_sale/sale.py? porque a sua solução não esta correta, pois o ["id"] do retorno do método read é do objeto (sale.shop) e não do fiscal_category_id. O Carlos Almeida já corrigiu isso e pediu o merge. O restante parece que esta tudo ok! assim que você fizer o revert do l10n_br_sale/sale.py da sua branch eu faço o merge.

Obrigado pela contribuição,

Renato Lima - http://www.akretion.com/pt-BR

review: Needs Fixing

« Back to merge proposal