Importação de XML e NFSe de Campinas

[danimaribeiro]

• [x] Melhorias no módulo de Manifesto.
• [x] Módulo de importação de xml
• [x] Módulo que integra o manifesto com a importação do xml
• [x] NFSe de Campinas em produção há alguns meses.
• [x] NFSe de São Paulo e reestruturação do módulo base NFSe.

[rvalyi]

So uma nota para dizer que eu pretendo revisar esse PR de importacao, so tou sem tempo para isso no momento mas vou tentar o mais cedo possivel.

[mileo]

Seria interessante se o problema fosse separado por módulo. É meio chato extrair isso, mas faz muito mais sentido na hora da revisão. Pretendo dar uma olhada em breve tb.... os módulos de importacao de XML estamos usando tb.

[rvalyi]

@mileo: concordo com vc seria interessante varios PR's, um por modulo.

@danimaribeiro pelo que eu vejo base_nfse nao depende do pysped. Por isso acho que vc poderia mandar um PR do base_nfse no repo OCA/l10n_brazil. Agora a gente tem que ver se vale a pena ser um modulo separado ou se bastaria fazer um merge dele no l10n_br_account_service talvez (adicionando a Trustcode na autoria como anunciado no caso). Bem talvez nao seja perfeito na 8.0 e fazemos melhor na 9.0 depois, vamos ver. Uma outra coisa seria interssante vc considerar o PR na OCA depois que o @renatonlima terminar esse PR https://github.com/OCA/l10n-brazil/pull/363 pois eu acho que vai facilitar muito as funcionalidades avancadas de nfe e nfse. O Renato esta terninando o PR ja ja...

[rvalyi]

@danimaribeiro na verdade nehnum desses modulos dependem do pysped. Mas por outro lado acho que so o base_nfse pode chegar na OCA a curto prazo (vamos dizer a 8.0). Primeiro que vamos ter que proceder por ordem e que isso pode levar um certo tempo.

Segundo que o modulo de importacao vai provavelemente ter que passar por um importante refator (pretendo falar dar mais retorno sobre isso; isso tb nao tira o merito do seu trabalho para atender uma necessidade hoje). A respeito dos modulos de campinas e sao paulo, com a pouca massa critica que temos na OCA achamos inviavel de comecar botar os templates de todos municipios hoje: nunca vai ter gente suficente para revisar e detalhes de mappeamentos vao acabar poluindo o trabalho estrutural feito ao nivel da OCA. Na hora da v10 com mais gente revisando nao digo, mas na v8 realmente nao pareceria realista.

Agora o modulo base_nfse, isolado ou como merge no l10n_br_account_service esse sim a gente consegue revisar e integrar ele de forma bem limpa a nivel da OCA.

Alias vamos bater um papo com o @mileo sobre o que a gente precisa fazer um pouco com a OCA e o pysped que ao nosso ver e onde e hoje o gargalho da localizacao a partir do momento que o emissor gratuito sera descontinuado. Mais retorno do @renatonlima sobre isso em breve... Focamos com urgencia no PR da separacao das visoes dos documentos fiscais e amanha ele ja estara pronto para ser revisado.

[rvalyi]

@danimaribeiro deixei um issue em aberto sobre o merge do base_nfse na OCA https://github.com/OCA/l10n-brazil/issues/392 Seria interessante vc comentar sobre porque o base_nfse depende do modulo 10n_br_account_product e se poderiamos fazer algum refator no modulo l10n_br_account para matar essa dependencia.

[danimaribeiro]

Basicamente ele depende do l10n_br_account_produc por causa do workflow da fatura. Como é o módulo 10n_br_account_product que modifica o workflow para a NFe, eu preciso herdar dele para colocar o workflow da NFSe.

Não sei se vale o trabalho tentar remover esta dependencia na 8.0.

[renatonlima]

@danimaribeiro neste PR na OCA https://github.com/OCA/l10n-brazil/pull/363 eu movi a mudança no workflow da invoice para o módulo l10n_br_account para ser usado pelo l10n_br_account_product e l10n_br_account_service

Eu ainda não olhei o seu código mas em breve eu devo fazer isso.

[mileo]

@danimaribeiro comando para vc separa os pr por modulo.

git checkout trust 8.0 git filter-branch --subdirectory-filter base_nfse -f git filter-branch -f --tree-filter 'mkdir -v base_nfse; git mv -k * base_nfse' HEAD git pull -r odoo-brazil 8.0 git checkout -b feature/filter-base_nfse git push origin feature/filter-base_nfse git checkout trust 8.0 git filter-branch --subdirectory-filter nfse_campinas -f git filter-branch -f --tree-filter 'mkdir -v nfse_campinas; git mv -k * nfse_campinas' HEAD git pull -r odoo-brazil 8.0 git checkout -b feature/filter-nfse_campinas git push origin feature/filter-nfse_campinas git checkout trust 8.0 git filter-branch --subdirectory-filter nfse_sao_paulo -f git filter-branch -f --tree-filter 'mkdir -v nfse_sao_paulo; git mv -k * nfse_sao_paulo' HEAD git pull -r odoo-brazil 8.0 git checkout -b feature/filter-nfse_sao_paulo git push origin feature/filter-nfse_sao_paulo