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

Melhorias para NF-e #74

Merged
merged 3 commits into from
May 10, 2024
Merged

Conversation

antoniospneto
Copy link
Contributor

@antoniospneto antoniospneto commented Mar 8, 2024

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

  • Essas consultas são desnecessaria e podem levar a um "Consumo Indevido" e ser penalizado pela sefaz com bloqueio.
  • O proprio manual da NF-e explica que não é correto ficar verificando assim a cada envio, apenas depois de uma falha no envio, o usuário deve usar o serviço de consultar o status para poder confirmar se o serviço realmente está fora do ar.
  • Diminui o tempo de resposta ao enviar a nota.
  • Quanto ao ficar consultado o status da nota também penso que é um exagero, pois sabemos que 99,99% dos casos a nota não vai tá enviada, se por acaso o usuário receber um aviso de duplicidade ao tentar enviar, ai sim, pode consultar a situação da nota.

2) Possibilidade de poder escolher a forma de transmissão sincrona.
commit: 1dc1c05

  • Não são todos os estados que aceitam o envio sincrono, exemplo: SP, GO, BA. (aceitam apenas para NFC-e)
  • a vantagem do modo sincrono é que o retorno do processamento da nota é bem mais rapido, não é preciso ficar aguardando para consultar o recibo.

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

@rvalyi
Copy link
Member

rvalyi commented Mar 16, 2024

@antoniospneto vc pode dar um rebase por favor agora que o #71 entrou?

@antoniospneto antoniospneto marked this pull request as draft March 18, 2024 12:59
@antoniospneto antoniospneto force-pushed the add-sinc-e-montar-proc branch 4 times, most recently from 984d6fb to c2d1612 Compare May 9, 2024 01:38
@antoniospneto antoniospneto force-pushed the add-sinc-e-montar-proc branch from c2d1612 to 19a5636 Compare May 9, 2024 01:39
@antoniospneto antoniospneto marked this pull request as ready for review May 9, 2024 01:41
Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rvalyi
Copy link
Member

rvalyi commented May 9, 2024

@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...)

@marcelsavegnago marcelsavegnago requested a review from mileo May 9, 2024 16:53
Copy link
Contributor

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antoniospneto antoniospneto merged commit 0499709 into erpbrasil:master May 10, 2024
11 checks passed
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.

5 participants