Merge lp://qastaging/~luciano-igtech/openerp.pt-br-localiz/l10n_br_inscr_est into lp://qastaging/openerp.pt-br-localiz/6.1

Proposed by Renato Lima - http://www.akretion.com
Status: Rejected
Rejected by: Renato Lima - http://www.akretion.com
Proposed branch: lp://qastaging/~luciano-igtech/openerp.pt-br-localiz/l10n_br_inscr_est
Merge into: lp://qastaging/openerp.pt-br-localiz/6.1
Diff against target: 1258 lines (+975/-82)
1 file modified
l10n_br_base/partner.py (+975/-82)
To merge this branch: bzr merge lp://qastaging/~luciano-igtech/openerp.pt-br-localiz/l10n_br_inscr_est
Reviewer Review Type Date Requested Status
Renato Lima - http://www.akretion.com Disapprove
Review via email: mp+94798@code.qastaging.launchpad.net

Description of the change

Adicionado o restante das validações faltantes no OpenERP, contribuição do luciano Schimer

To post a comment you must log in.
Revision history for this message
Renato Lima - http://www.akretion.com (renatonlima) wrote :

É importante adicionar um arquivo para testar todas as funções para ter certeza que todas as rotinas de validações estão corretas, para não gerar problemas durante o cadastro de um parceiro no OpenERP.

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

Olá Luciano,

Como o Leonardo baixou a sua branch e fez um refactoring dos métodos eu vou rejeitar este pedido e vamos analisar o merge do refactoring

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

Foi refatorado esse medido de merge e gerado outro pedido de merge

review: Disapprove

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.