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

[FIX] Não carregue novamente as mensagens fiscais da nf-e #3254

Draft
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

mileo
Copy link
Member

@mileo mileo commented Jul 31, 2024

Fecha o issue #3253

@OCA-git-bot
Copy link
Contributor

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

@rvalyi
Copy link
Member

rvalyi commented Jul 31, 2024

Fecha o issue #3253

so para deixar claro @mileo na OCA quando tem um bug que a pessoa corrige logo depois num PR, o pessoal nao costuma abrir um issue e explica direitamente no PR o que o PR corrige, que assim faz menos "barulho" parasita no projeto. Se abre um issue apenas quando nao se tem o PR para corrigir ou se eh previsivel que vai demorar muito porque a correçao eh complexa. Nao sei se isso ta escrito em algum lugar mas de fato eh que o pessoal faz e que tem dado certo.

@antoniospneto
Copy link
Contributor

antoniospneto commented Jul 31, 2024

Quando um documento fiscal é confirmado e passa para o estado Aguardando envio, os comentários manualmente removidos são inseridos novamente.

Qual o outro momento que os comentários são carregados da operação fiscal para dentro do documento fiscal?
Pois pelo testes isso parece ter gerado uma regressão, o teste falhou na geração da nfe pois o coméntario não foi carregado da operação fiscal.

Eu entendo que podemos ter um problema aqui, a funcionalidade dos comentário / obeservações deixa a desejar as veses, mas é preciso avaliar melhor.

Se puder detalhar certinho o caso de uso onde o problema acontece.

@antoniospneto antoniospneto marked this pull request as draft July 31, 2024 13:40
@mileo
Copy link
Member Author

mileo commented Jul 31, 2024

Fecha o issue #3253

so para deixar claro @mileo na OCA quando tem um bug que a pessoa corrige logo depois num PR, o pessoal nao costuma abrir um issue e explica direitamente no PR o que o PR corrige, que assim faz menos "barulho" parasita no projeto. Se abre um issue apenas quando nao se tem o PR para corrigir ou se eh previsivel que vai demorar muito porque a correçao eh complexa. Nao sei se isso ta escrito em algum lugar mas de fato eh que o pessoal faz e que tem dado certo.

@rvalyi não tinha a intenção de corrigir na hora por isso abri o issue, mas ao investigar a solução resolvi propor o PR.

@mileo
Copy link
Member Author

mileo commented Jul 31, 2024

Quando um documento fiscal é confirmado e passa para o estado Aguardando envio, os comentários manualmente removidos são inseridos novamente.

Qual o outro momento que os comentários são carregados da operação fiscal para dentro do documento fiscal? Pois pelo testes isso parece ter gerado uma regressão, o teste falhou na geração da nfe pois o coméntario não foi carregado da operação fiscal.

Eu entendo que podemos ter um problema aqui, a funcionalidade dos comentário / obeservações deixa a desejar as veses, mas é preciso avaliar melhor.

Se puder detalhar certinho o caso de uso onde o problema acontece.

Vou verificar essa regressão com calma.

Mas o bug surgiu pois um cliente tem uma mensagem que em determinada nota excede os 500 caracteres permitidos, então ele deseja remover a mensagem, mas mesmo removendo-a, ela volta a ser adicionada ao confirmar o edoc.

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