Raphaël Valyi
Raphaël Valyi
/ocabot merge nobump
@mileo @antoniospneto esse PR é bom o @renatonlima revisar porque até onde me falou não achou legal... @renatonlima consegue se posicionar sobre esse PR? Já faz muito tempo...
pessoal, o @renatonlima tava numa corrida importante até hoje para um cliente que vai migrar da 12 para a 16. Vou ver com ele se vai aliviar agora. Eh que...
Eh que não é um corretivo simples de 10 linhas, toca 23 arquivos, adiciona 442 linhas e tira 123. Nisso eu acho indispensável o Renato que fez uns 80% do...
"solução alternativa" de 10 ou 20 linhas para problema critico é tranquilo fazer merge. O problema é um problema de "entropia": não da para experimentar com PR de 400 linhas...
Alias @mileo vc quase não fez mais codigo os 3 ultimos anos porque vc escolhou de delegar para desenvolvedores iniciantes. Mas ainda assim vc sempre insiste para querer dar palpite...
> Como ta a situação dessa proposta @rvalyi ? Já da pra testar de forma concreta? > vou subir uns testes em breve...
@felipemotter @antoniospneto @marcelsavegnago @renatonlima @mbcosta @mileo The PR is ready now for review now. I made e few changes and added a complete test suite (it's inside the test_account_move_lc.py file...
Eu atualizei o PR de importação da NFe com esses 4 commits https://github.com/OCA/l10n-brazil/pull/2781#issuecomment-2012320590
@mileo não era isso que vc precisava aqui na sua RFC https://github.com/OCA/l10n-brazil/issues/894 ? ai que tal testar...