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

Otimização na função findConversation em chatWootClient.ts #1330

Merged
merged 2 commits into from Jun 24, 2023

Conversation

miguelurtado
Copy link
Contributor

Buscar somente as mensagens do contato e não todas as mensagens. Em alguns casos com um número alto de mensagens, buscar todas as mensagens, causa erro 500.

Miguel Urtado added 2 commits June 23, 2023 14:01
@miguelurtado
Copy link
Contributor Author

Essa correção também evita a duplicação de mensagens no Chatwoot. Pois, ao dar erro na busca das mensagens o fluxo segue e cria uma nova.

@icleitoncosta
Copy link
Collaborator

Opa @miguelurtado, você fez todos os testes necessários?

Infelizmente não consigo testar com chatwoot aqui.

@miguelurtado
Copy link
Contributor Author

Opa... fiz os testes na prática. Levantei um ambiente local aqui e conectei no meu chatwoot online. Aí na versão atual estava dando erro 500 devido a quantidade de mensagens. Quando otimizei para trazer somente as mensagens do contato em questão, que no caso já foi consultado no chatwoot, aí da tudo certo.

@icleitoncosta icleitoncosta merged commit d06bd67 into wppconnect-team:main Jun 24, 2023
2 checks passed
@miguelurtado
Copy link
Contributor Author

Legal que deu certo o merge. Alguma previsão para um release com essa otimização?

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.

None yet

2 participants