Raphaël Valyi
Raphaël Valyi
Ola @marcelsavegnago valeu pela limpeza, porem revisando eu vi um problema que eu aproveitei para corrigir aqui https://github.com/Escodoo/l10n-brazil/pull/47 cc @mbcosta @renatonlima @netosjb
bom, já vou fazer o merge desse para começar e depois a gente vê dos outros pontos... /ocabot merge minor
valeu pelo review @felipemotter eu fiz as correçoes, se puder revisar de novo eu agradeço. cc @marcelsavegnago @netosjb @renatonlima @mbcosta
a principio essa semana o @mbcosta deveria poder dar uma olhada, vamos apenas aguardar ele dar um retorno...
os testes que falham falham no modulo payment_pagseguro por conta dos testes Javascript. Não tem a ver com minha alteração aqui...
@mileo quel tal nesse caso que vc menciona a gente acrescentar alguma propriedade na empresa. E ai se a empresa for optar por esse sistema (caso raríssimo), ai no partner_id...
@mileo tipo algo assim https://github.com/akretion/l10n-brazil/commit/aa18f452e593090c15bd9d6cc336d3b863c64b3e ?
Mas é vdd que no caso raríssimo de uma empresa como o @mileo mencionou ela poderia até preferir da forma como tá hoje... Vc chegou a ver essa minha proposta ...
pessoal vou cortar o commit em 2: 1 por modulo...
lembrando que qdo chegar ao nivel de modulos que implementam documentos fiscais como da NFe, tem uma porrada de validações de REGEXP que sera feita pelos validadores do generateDS. Mas...