Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0] Melhorias na transmissão da NFE #2945

Closed
wants to merge 2 commits into from

Conversation

antoniospneto
Copy link
Contributor

@antoniospneto antoniospneto commented Mar 8, 2024

Trabalho em progresso.

Melhorias propostas nessa PR:

  1. Suporte ao envio da nfe no método sincrono.

  2. Melhora o processo do envio quando é assincrono.
    É possivel consultar o recibo novamente em caso de falha, os dados da nota enviada não são perdidos.

  3. Implementa o wizard de consultar situação para a NF-e.

  4. Mais algumas outras pequenas correções de usabilidade.

Depende das melhorias proposta nas libs externas:

Nota: Pretendo depois quebrar em PRs menores.

@OCA-git-bot
Copy link
Contributor

Hi @rvalyi, @renatonlima,
some modules you are maintaining are being modified, check this out!

@mileo
Copy link
Member

mileo commented Mar 8, 2024

Muito bom!

@antoniospneto
Copy link
Contributor Author

antoniospneto commented Mar 11, 2024

Pessoal, ontem eu organizei a PR erpbrasil/erpbrasil.edoc#71 e está pronta para revisão, quem puder revisar lá eu agradeço, depois de aprovado as PRs no erpbrasil.edoc vou dar continuidade nos trabalhos aqui, valeu!

@rvalyi
Copy link
Member

rvalyi commented May 10, 2024

@antoniospneto entraram todas dependencias no nfelib e erpbrasil.edoc, se puder atualizar o PR para a gente ver melhor como fica aqui seria top...

@antoniospneto
Copy link
Contributor Author

antoniospneto commented May 10, 2024

os testes estão passando, mas eu vou adicionar mais alguns para evitar regressões e melhorar a cobertura dos testes
vou também quebrar os commits um pouco pra ficar mais explicativo

@antoniospneto
Copy link
Contributor Author

substituido por #3081

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants