← Back to team overview

openerp-brazil-team team mailing list archive

Re: lp:~santagada-proge/openerp.pt-br-localiz/l10n_br_inscr_est_refatorada into lp:openerp.pt-br-localiz

 

Review: Disapprove

Existe 6 bugs no refactoring, e não existe teste, eu criei um novo refactoring em uma nova branch https://code.launchpad.net/~openerp-brazil-core-team/openerp.pt-br-localiz/l10n_br_inscr_est_refatorada_v2 corrigindo esses bugs e coloquei testes com em media 5 inscrições estaduais de todos os estados, na nova branch eu dei permissão aos commiters da localização para corrigir e testar, pois as inscrições estaduais de MG e MT ainda não estão sendo validadas e também seria bom remover algumas alterações feitas em outras linhas que não faz parte da validação da inscrição estadual e deixar em outra branch

Este pedido de merge esta sendo rejeitado e após a correção dos bugs deve ser criado outro pedido de merge com a nova branch.


Renato Lima
-- 
https://code.launchpad.net/~santagada-proge/openerp.pt-br-localiz/l10n_br_inscr_est_refatorada/+merge/95037
Your team OpenERP Brazil Team is subscribed to branch lp:openerp.pt-br-localiz.


References