Skip to content

DANFE: Adiciona marca d'água "cancelada" em DanfeConfig para caso a nfe seja cancelada.#93

Merged
antoniospneto merged 2 commits intoEngenere:mainfrom
PedroJoMa:danfe-new-config-cancelled-nfe
May 8, 2025
Merged

DANFE: Adiciona marca d'água "cancelada" em DanfeConfig para caso a nfe seja cancelada.#93
antoniospneto merged 2 commits intoEngenere:mainfrom
PedroJoMa:danfe-new-config-cancelled-nfe

Conversation

@PedroJoMa
Copy link
Contributor

@PedroJoMa PedroJoMa commented Apr 29, 2025

O que foi feito

  • Foi feito a implementação de uma nova configuração para Danfes, caso a nota fiscal seja cancelada, temos uma flag que altera a marca d'água da Nfe.
  • Foram feitos testes automatizados para certificar a criação de arquivos pdf com as alterações da marca d'água.

Imagens

NFE cancelada com Ambiente de Produção (tpAmb = 1)

display_cancelled_with_production

NFE cancelada sem Ambiente de Produção (tpAmb != 1)

display_cancelled_without_production

@PedroJoMa PedroJoMa changed the title DANFE: Adiciona tag "cancelado" em DanfeConfig para caso a nfe seja cancelada. DANFE: Adiciona marca d'água "cancelado" em DanfeConfig para caso a nfe seja cancelada. Apr 29, 2025
@PedroJoMa PedroJoMa changed the title DANFE: Adiciona marca d'água "cancelado" em DanfeConfig para caso a nfe seja cancelada. DANFE: Adiciona marca d'água "cancelada" em DanfeConfig para caso a nfe seja cancelada. Apr 29, 2025
font_size = 60
if is_production_environment and is_protocol_available:
return
if self.watermark_cancelled:
Copy link
Contributor

@antoniospneto antoniospneto Apr 29, 2025

Choose a reason for hiding this comment

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

hum, mas se for ambiente de produção e tiver um protocolo de autorização informado não vai imprimir a marca da agua "cancelado" pois o programa vai sair da função na linha 494, mesmo que watermark_cancelled esteja definido como true, vc tem que fazer essa verificação antes.

Recomendo por no arquivo nfe_cancelled_without_production_environment.xml a tag com o protocolo de autorização pro teste ficar mais fiel.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Olá, fiz as alterações conforme discutido.

Copy link
Contributor

@CristianoMafraJunior CristianoMafraJunior left a comment

Choose a reason for hiding this comment

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

@PedroJoMa parabéns pelo trabalho por mim ta ok
só consegue juntar todos esses 4 commits em um?

Copy link
Contributor

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

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

Um último detalhe, quando você imprime o "cancelada - sem valor fiscal"
Você acabou precisando diminuir a altura da fonte, com isso você precisa ajustar as formulas logo a baixo que centraliza o texto, veja como na fonte menor fica fora do centro:

image

@CristianoMafraJunior
Copy link
Contributor

CristianoMafraJunior commented May 2, 2025

@PedroJoMa uma coisa que também seria interessante é colocar essa questão na documentação de DANFE cancelada

@PedroJoMa PedroJoMa requested a review from antoniospneto May 7, 2025 12:46
@PedroJoMa
Copy link
Contributor Author

@CristianoMafraJunior fiz as devidas correções no alinhamento do texto, farei a contribuição na documentação da DANFE em breve

@antoniospneto
Copy link
Contributor

@CristianoMafraJunior fiz as devidas correções no alinhamento do texto, farei a contribuição na documentação da DANFE em breve

@PedroJoMa show, agora ficou bem bom com o ajuste!

@antoniospneto
Copy link
Contributor

@PedroJoMa consegue fazer um squash nos commits também?

@antoniospneto
Copy link
Contributor

@PedroJoMa como houve atualização na branch principal nesse meio tempo, você vai precisar fazer um rebase na sua branch com a branch principal, para resolver os conflitos.

@PedroJoMa PedroJoMa force-pushed the danfe-new-config-cancelled-nfe branch from dff529d to 22b9c79 Compare May 8, 2025 15:03
@PedroJoMa
Copy link
Contributor Author

@antoniospneto Squash e atualização da documentação realizada!

Copy link
Contributor

@CristianoMafraJunior CristianoMafraJunior left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

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

@PedroJoMa

Obrigado por atender as solicitações, parabéns pelo trabalho.

@antoniospneto antoniospneto merged commit 2559d50 into Engenere:main May 8, 2025
6 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.

3 participants