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

Implementa DANFE HTML #1422

Closed
wants to merge 1 commit into from
Closed

Implementa DANFE HTML #1422

wants to merge 1 commit into from

Conversation

DSBRBrasil
Copy link
Contributor

Segue para analise implementacao danfe em HTML modelo 55 sem nenhuma dependencia de software externo

Copy link
Contributor

@marcosgerene marcosgerene left a comment

Choose a reason for hiding this comment

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

Iniciei a revisão mas parei porque são muitos arquivos alterados sem necessidade.

Seria possível (sei que é um commit grande) isolar somente a adição, ou seja, remover desse PR o que diz respeito a outros projetos como o downgrade da versão do .NET nos App.Teste?

Obrigado pela contribuição, te garanto que vai ser muito útil para a comunidade!

Copy link
Contributor

Choose a reason for hiding this comment

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

002

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Não alterei arquivos existentes, foi apenas o projeto NFe.Danfe.Nativo2.
Fiz o fork, baixei o projeto e pronto.
Podem criar um branch e orientar ?

Copy link
Contributor

Choose a reason for hiding this comment

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

002

Copy link
Contributor Author

@DSBRBrasil DSBRBrasil Jun 10, 2023

Choose a reason for hiding this comment

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

Descarte qualquer alteração de arquivo ! apenas foque no projeto NFe.Danfe.Nativo2.
Sugiro que deixem o projeto o mais isolado possível dos demais dá pra colocar ele até no nuget somente.
O da NFCe acredito que em meados de setembro posso estar mandando para vcs.

@robertorp
Copy link
Contributor

Alterou a versão dos projetos de testes apenas hehe , não ta dificil de mudar , vou olhar também seu projeto , o que você cirou foi o nativo2 para o html né ?

@DSBRBrasil
Copy link
Contributor Author

Alterou a versão dos projetos de testes apenas hehe , não ta dificil de mudar , vou olhar também seu projeto , o que você cirou foi o nativo2 para o html né ?
@robertorp Me lembrei que eu ja tinha o fork desse projeto há anos, e acredito que uma alteração antiga pode ter ido nesse pr, então pode descartar qualquer alteração em outros projetos. O único que deve ser foco de analise é o NFe.Danfe.Nativo2
Como funciona?

  1. Para nao quebrar o layout a pagina inicial comporta:
    1.1 - 7 linhas se no pior caso houver mais de 800 caracteres entre os 7 primeiros registros de produtos, oque significa que há descrições com 120 caracteres permitidos em xProd
    1.2 - 12 linhas se no melhor caso houver até 800 caracteres entre as 12 primeiras linhas do xml

  2. Para nao qebrar o layout das paginas subsequentes à primeira:
    2.1 - 24 linhas se no pior caso houver uma contagem de mais de 3000 caracteres nas 24 primeiras linhas
    2.2 - 50 linhas de no melhor caso houver uma contagem de até 3000 caracteres nas 50 primeiras linhas

Resumo: Após vários testes o layout se comportou muito bem, forcei mais de 100 itens (produtos) oscilando entre 20 a 120 caracteres. Quando em produção a maior parte das danfes continham de 1 a 3 itens e poucas no máximo com 19 itens

image
image
image
Peço ajuda dos analistas para melhorem o codigo fonte

@robertorp
Copy link
Contributor

@DSBRBrasil olha esse branch que fiz pegando teu pull as partes que realmente importam e alterando somente onde necessário, você se importa de min fazer merge desse pull que fiz ? Digo isso porque por algum motivo o seu branch alterou muita coisa do Zeus , digo os projetos dele
https://github.com/ZeusAutomacao/DFe.NET/tree/ra/feat-danfe-html-merge

@DSBRBrasil
Copy link
Contributor Author

@DSBRBrasil olha esse branch que fiz pegando teu pull as partes que realmente importam e alterando somente onde necessário, você se importa de min fazer merge desse pull que fiz ? Digo isso porque por algum motivo o seu branch alterou muita coisa do Zeus , digo os projetos dele https://github.com/ZeusAutomacao/DFe.NET/tree/ra/feat-danfe-html-merge

Opa !!! Pode fazer oque for necessário. Eu errei ao pegar um fork meu que estava alterado e mandar o PR ( eu tinha esquecido que havia alterado para construção de uma outra solução com base no Zeus).
Assim que vc terminar, me avisa para que eu, a partir dessa nova branch correta criada por você, realize um ajuste no layout necessário quando for usar um componente de conversao para PDF (a marca d'agua nao estava sendo exibida ao converter para PDF usando o iTExtPDF

@robertorp
Copy link
Contributor

robertorp commented Jun 14, 2023

@DSBRBrasil já está no master =) tem até app teste , obrigado , isso é muito bom esse danfe ,
vou dar close aqui , vlw

@robertorp robertorp closed this Jun 14, 2023
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