-
Notifications
You must be signed in to change notification settings - Fork 19
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
Melhorias para NF-e #74
Melhorias para NF-e #74
Conversation
1dc1c05
to
bcbc0fc
Compare
@antoniospneto vc pode dar um rebase por favor agora que o #71 entrou? |
984d6fb
to
c2d1612
Compare
c2d1612
to
19a5636
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@mileo sendo que é a KMEE que fez a maioria dessa lib, seria bom a gente ter um feedback de vcs para não prejudicar algum cliente de vcs com a mudança... (digamos que seria o mesmo tipo de respeito que a gente exige nos módulos que a gente lidera... nada de muito maluco no mundo do open source...) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Essa PR tem como proposta adicionar 3 melhorias para as NF-e
1) Desativa por padrão a "consulta do status do serviço" e a "consulta da situação da nota" antes enviar.
commit: 510bcd0
2) Possibilidade de poder escolher a forma de transmissão sincrona.
commit: 1dc1c05
3) Adiciona uma função auxiliar para montar o arquivo final da NF-e.
é a tag nfeProc, essa tag inclui a nfe enviada mais o protocolo de autorização recebido da sefaz.
Para o emitente a sefaz nunca retorna o xml da nfe completo, apenas o protocolo de autorização.
A sefaz espera que o sistema emissor tenha guardado bem o xml da nota enviada :)
commit a4407c5
Depende da PR da refotoração do CI e testes: Refatoração do CI, Github Actions, Pre-commit e Testes #71