[14.0][REF] Impressão da DANFE a partir da lib BrazilFiscalReport #3068
Conversation
|
Hi @renatonlima, @rvalyi, @felipemotter, @mbcosta, |
|
Entrando em prod no nosso cliente piloto. |
|
@antoniospneto, @felipemotter Recentemente esse ponto confundiu o @rodmad85 por examplo e a gente teve que gastar tempo para re-explicar porque nao tem DANFE na CI... |
Então agora temos DANFE no runboat :D |
|
Eu gosto bastante da ideia de tirar o peso do libre office e também a complexidade de manutenção(diffs de PRs) do py3o. Como esta é uma lib importante eu vejo com bom olhos colocar ela sobe o guarda-chuva da organização do erpbrasil, percebo que integrar esta lib ao erpbrasil facilitaria o acesso e uso das dependências da localização, simplificando a manutenção e a colaboração entre as equipes. O lib com certeza pode ser mantida separada, assim como a nfelib. No entanto, considero que centralizar nossos recursos sob o mesmo guarda-chuva não apenas aumentaria a confiabilidade e a governança do projetos, mas também reduziria a complexidade para os novos usuários que se juntam à localização. No longo prazo eu vejo que a ideia é remover o erpbrasil.edoc.pdf Vejo que vale uma atualização na pagina do https://erpbrasil.github.io/ citando a Engenere e a Escodoo tb. Alem de atualizar algumas referências das libs. Isso poderia alinhar nossas estratégias mantendo a autonomia do projeto. |
|
valeu @antoniospneto parabéns pelo trabalho, apenas uma questão inicial vi que o projeto foi baseado em outro https://github.com/edsonbernar/nfe_utils e que vocês colocaram a referencia sobre isso, mas houve alguma tentativa de contato com o autor? Para saber se ele teria algum interesse em passar a colaborar em um projeto único ou no repo dele ou no de vocês ou em algum outro, erpbrasil como @mileo está sugerindo, já que vocês implementaram documentação e testes que é algo que no original está apenas no ROADMAP |
|
Isso, o projeto foi criado a partir do fork do nfe_utils, mas, apesar disso, foi todo refatorado. Se for comparar o código, o diff é quase de 100%. Como as alterações foram muito significativas, decidimos manter o nosso próprio fork, mas o autor original do projeto foi contatado sim. Caso haja interesse, ele pode integrar as alterações. A Engenere pretende se manter como mantedora do projeto, o que garante que a gente se comprometa bastante com o projeto. Acreditamos que, ao manter a gestão centralizada, podemos assegurar a continuidade e a qualidade do desenvolvimento. Nosso objetivo é proporcionar uma ferramenta robusta e confiável para a comunidade, e estamos comprometidos em fornecer suporte e melhorias contínuas. No momento, não há interesse em repassar para outro repositório, como o erpbrasil, mas estamos abertos a colaborações e contribuições que possam beneficiar o projeto. cc @mileo |
|
Aproveitando para comentar apenas sobre essa questão: a partir do momento que a P&D da nova lib BrazilFiscalReport foi assumida apenas pela Engenere, não vejo problema em manter a lib num repo da Engenere. Falando em governancia é algo muito mais logico do que pessoas aleatórias que nada fizeram nessa P&D comecar a entrar no meio e dar palpite de engenheiro da obra pronta ou permitir a estagiários que nunca emplacariam PR algum na OCA de comecar a adicionar funcionalidades adicionais aleatórias... Hoje a Engenere certamente tem contribuições no OCA/l10n-brazil com qualidade acima da media então não vejo risco com isso... Caso algum dia a Engenere abandonasse a manutenção da lib, ai sim alguma org como erpbrasil seria natural para manter um fork ativo, assim como foi feito com erpbrasil.edoc qdo o pysped foi abandonado pelo próprio autor. Lembrando tb que a situação foi diferente com as libs erpbrasil.base ou erpbrasil.assinatura onde foi uma P&D compartilhada da Kmee e da Akretion ai sim fez sentido botar numa org comum. Agora tb essa org erpbrasil, tem muita coisa lá que é bom nem se orgulhar muito não... Se alguém cascudo da OCA olhar o que tem nessa erpbrasil, capaz de sair com medo em vez de achar bacana... Por fim eu acho que a gente so tem que agradecer a Engenere de ter encabeçado o desafio de fazer essa lib. Pois a lib erpbrasil.edoc.pdf andava meio abandonada pela Kmee e usava o engine py3o que tinha sido abandonado pelo proprio autor, a XCG. Quebrou um bom galho alguns anos (depois da gente ter usado Aeroo e Geraldo...) mas tava um ponto muito fraco das dependencias do OCA/l10n-brazil. Segue um print dos últimos PRs no erpbrasil.edoc.pdf: mal teve review da Kmee: Acho complicado nem conseguir cuidar da propria lib e querer fiscalizar o trabalho dos outros... Entendo que a Kmee é co-autor do módulo l10n_br_nfe e por isso pode exigir de nao sair prejudicada pelo PR. Porem aqui o PR não é invasivo e mantem a possibilidade de imprimir o DANFE usando erpbrasil.edoc.pdf. Nao vejo grande problema com isso... |
|
@antoniospneto pode fazer um rebase por favor ? |
|
Pessoal, O @felipemotter fez o rebase e tbm incluimos a possibilidade de fazer algumas configurações na DANFE:
A possibilidade de exibir os campos PIS e COFINS foi a pedido do @renatonlima |
antoniospneto
force-pushed
the
14.0-nfe-danfe
branch
2 times, most recently
from
9160ac2
to
a0fe5a9
Compare
|
Após o rebase com a branch principal os testes não estão mais passando, vou verificar o motivo, por enquanto vou deixar a PR em rascunho. |
antoniospneto
force-pushed
the
14.0-nfe-danfe
branch
from
a0fe5a9
to
5cc0d87
Compare
|
@OCA/local-brazil-maintainers Pronto para revisão de volta. |
Ai mais tarde quando a lib BrazilFiscalReport estiver bem testada a gente pode remover a dependencia do eprbrasil.edoc.pdf se for do agrado de todo mundo, simplificando o código. |
|
This PR has the |
antoniospneto
force-pushed
the
14.0-nfe-danfe
branch
from
5cc0d87
to
a359bee
Compare
|
(botando merge major pela nova dependencia) /ocabot merge major |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 1016729. Thanks a lot for contributing to OCA. ❤️ |
Adiciona opção de imprimir a DANFE a partir da biblioteca brazilfiscalreport
A biblioteca a ser usada pode ser definido na configuração da empresa:
ainda é possivel configurar para imprimir a DANFE usando o jeito antigo (erpbrasil usando o libreoffice) é bom ter essa opção até que se confirme que a DANFE no BrazilFiscalReport se confirme estavel.
Agora é possivel também imprimir a DANFE usando os menus nativos de relatórios do Odoo:
Foi implementando tando no modelo
account.movequantol10n_br_fiscal.documentExemplos de DANFE geradas podem ser vistas no repositório da biblioteca: https://github.com/Engenere/BrazilFiscalReport/tree/main/tests/generated/danfe