search

OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
236 stars 244 forks source link

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

Open mileo opened 1 month ago

mileo commented 1 month ago

Fecha o issue #3253

OCA-git-bot commented 1 month ago

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

rvalyi commented 1 month ago

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 commented 1 month ago

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.

mileo commented 1 month ago

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 commented 1 month ago

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.