Skip to content

Alteração da função wait_for_download() em botcity>web>bot.py #108

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gabrielms-castro
Copy link

Boa tarde, prezado.

Eu estava com um problema numa automação em que precisava utilizar o wait_for_download() e percebi que estava dando erro em todas minhas tentativas.

Olhando um pouco a função na biblioteca, experimentei "debugar" a função com print() mesmo e percebi que o if não estava inicializando e sempre caia no TimeoutException.
Experimentei remover o and self.headless do if na linha 1218 e funcionou perfeitamente na minha automação.

Poderiam testar e ver se faz sentido?

Obrigado, pela sua atenção.

Att,
Gabriel Castro

@kayqueGovetri kayqueGovetri self-requested a review September 9, 2024 21:50
@kayqueGovetri
Copy link
Contributor

@gabrielms-castro

Olá Gabriel, obrigado por contribuir com este pull request.

Preciso saber coisas mais especificas sobre o problema para averiguarmos e testarmos do nosso lado também, que seriam:

  • Versão do sistema operacional.
  • Navegador utilizando e versão.
  • Versão do pacote da botcity.
  • Traceback do erro.

Seria interessante também abranger mais o que você estava tentando fazer para termos uma visão melhorar do problema.

Estarei esperando sua resposta.


Hello, Gabriel; thank you for contributing to this pull request.

I need to know more specific details about the issue so we can investigate and test it on our side as well, which would be:

  • Operating system version.
  • Browser and version used.
  • Botcity package version.
  • Error traceback.

It would also be helpful to provide more context about what you were trying to do so we can better understand the problem.

I’ll be waiting for your response.

@gabrielms-castro
Copy link
Author

gabrielms-castro commented Sep 9, 2024 via email

@kayqueGovetri
Copy link
Contributor

kayqueGovetri commented Sep 12, 2024

Olá @gabrielms-castro ,

não estou conseguindo visualizar as imagens, mas consegui entender a situação, gostaria de pedir para que você tentasse algumas abordagens diferentes.

  • Instale a versão da branch main pelo seu requirements.txt:
    • botcity-framework-web @ git+https://github.com/botcity-dev/botcity-framework-web-python@main

Essa versao tem algumas correções e utiliza o selenium 4, pode ser que tenha que mudar algumas coisas em seu código após essa modificação.
Se conseguir testar dessa forma e dar um feedback. Se possível tente enviar as imagens corretamente também que já melhora nossa visualização.

Estarei esperando sua resposta.


I can't view the images, but I understand the situation. I would like to ask you to try a few different approaches.

Install the version from the main branch via your requirements.txt file:
botcity-framework-web @ git+https://github.com/botcity-dev/botcity-framework-web-python@main
This version includes some fixes and uses Selenium 4, so you might need to adjust your code after this change. If possible, test it this way and provide feedback. Also, please try to send the images correctly to improve our visibility.

I'll be waiting for your response.

@gabrielms-castro
Copy link
Author

gabrielms-castro commented Sep 12, 2024 via email

@kayqueGovetri
Copy link
Contributor

Ola @gabrielms-castro,

nos testes unitários que possuimos a função está funcionando na ultima versão do windows com o Chrome, consegue me passar as imagens novamente (elas estão com erro.)


Hello @gabrielms-castro,

In the unit tests we have, the function is working on the latest version of Windows with Chrome. Could you please send me the images again (they are showing an error)?
image

@gabrielms-castro
Copy link
Author

gabrielms-castro commented Sep 12, 2024 via email

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.

2 participants