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

Nota paulistana com instabilidade #3530

Open
mileo opened this issue Dec 5, 2024 · 6 comments
Open

Nota paulistana com instabilidade #3530

mileo opened this issue Dec 5, 2024 · 6 comments
Labels

Comments

@mileo
Copy link
Member

mileo commented Dec 5, 2024

O webservice da nota paulistana sempre da timeout ou não responde na primeira vez.

Isso tem causado bastante problemas nos clientes.

Vejo algumas estretegias possíveis:

  1. Aumentar o timeout
  2. Usar um sistema de filas para tentar novamente e ser totalmente assíncrona. (Envio e consulta separados)
@mileo mileo added the bug label Dec 5, 2024
@rvalyi
Copy link
Member

rvalyi commented Dec 5, 2024

não sei se seria legal depender do queue_job no mesmo modulo basico. Mas sem ser isso, um simples retry de algumas vezes dentro da
o mesmo query pro Odoo não resolveria? Vale a pena experimentar. Caso o servidor da cidade não costuma ficar fora do ar por minutos pode resolver. Caso contrário ai so fila mesmo...

@antoniospneto
Copy link
Contributor

acho que o mais interessante seria o queue_job em um módulo separado, se ficar segurando a transação no odoo com timeout maior ou retry pode dá problema de concorrencia no banco de dados

@rvalyi
Copy link
Member

rvalyi commented Dec 6, 2024

talvez é possível dar um reset do cursor da transação antes do retry para evitar os erros de "transaction isolation". Eu acho que eu vi isso em alguns lugares...

Mas finalizar o modulo queue_job generico seria bom tb.

@marcelsavegnago
Copy link
Member

Eu tinha feiyto esta PR com base em uma iniciativa da Kmee.. se a Kmee quiser dar andamento, talvez tenha que mudar o nome do modulo e tem uns dois itens que eu havia sinalizado na PR que precisaria implementar. Enfim, quem quiser assumir, comecamos mexer mas nem usamos este modulo hoje.

#1887

@antoniospneto
Copy link
Contributor

antoniospneto commented Dec 6, 2024

Eu tinha feiyto esta PR com base em uma iniciativa da Kmee.. se a Kmee quiser dar andamento, talvez tenha que mudar o nome do modulo e tem uns dois itens que eu havia sinalizado na PR que precisaria implementar. Enfim, quem quiser assumir, comecamos mexer mas nem usamos este modulo hoje.

#1887

Eu até cheguei por em produção uma vez, mas o cliente não gostou muito, pq tinha q dar f5 pra ver ser a nota foi ou não emitida kkkk ai sei lá teria que pensar em algo pra que o javascript recebesse uma notificação e atualizasse a pagina, na epóca eu desisti e não fiquei pensando muito.

No fim o que o cliente quer é a nota ser transmitida no mesmo segundo, mas é complicado quando as prefeitura não colaboram..

@rvalyi
Copy link
Member

rvalyi commented Dec 6, 2024

Eu tinha feiyto esta PR com base em uma iniciativa da Kmee.. se a Kmee quiser dar andamento, talvez tenha que mudar o nome do modulo e tem uns dois itens que eu havia sinalizado na PR que precisaria implementar. Enfim, quem quiser assumir, comecamos mexer mas nem usamos este modulo hoje.
#1887

Eu até cheguei por em produção uma vez, mas o cliente não gostou muito, pq tinha q dar f5 pra ver ser a nota foi ou não emitida kkkk ai sei lá teria que pensar em algo pra que o javascript recebesse uma notificação e atualizasse a pagina, na epóca eu desisti e não fiquei pensando muito.

Pensa muito mais não https://github.com/OCA/web/tree/16.0/web_notify

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants