Adiciona parametro de timeout para serviços de cep #206
Conversation
There was a problem hiding this comment.
Sobre os pontos levantados:
1 - Na verdade tava sim mas tava meio estranho pq esquecemos algumas coisas pra trás dado a correria que foi remover o proxy. Perdão a bagunça mestre e MUITO obrigado por apontar isso! 😢 🤩
2 - Boooa mestre, curti o default 30 acho que faz mais sentido que infinito mesmo! 🤘
3 - Quanto a testes acho que é interessante sim criar uns testes, acho que da pra fazer alguns testes legais, por exemplo passando apenas um provider e um timeout baixo tipo de 100 milissegundos pra ver se conseguia simular 😬
Posso te ajudar nos testes se precisar mestre! ^^
Sobre o PR:
Muito boa a proposta e a implementação manão!!
Fiz um teste local e tá funcionando perfeitamente! 😍 🚀
Deixei só alguns comentários de dúvidas e sugestões mas são mais focados em complementar o PR afinal a funcionalidade já parece 100%! 😬 ❤️ 🚀
lucianopf
dismissed
their stale review
Miss click, na verdade era só um comentário
|
Valeu @lucianopf ! Vou rever os pontos que você levantou e atualizo aqui o PR nos próximos dias |
|
Feito @lucianopf |
|
@lucianopf Opa! tá a um tempinho parado esse, essa feature ainda é relevante? |
|
Meeestre é sim!! Mil perdões, esqueci de adicionar os PRs do cep-promise no Slack e tão todos parados 😭 Hoje ainda reviso ele!! |
lucianopf
changed the title
There was a problem hiding this comment.
Primeiramente parabéns pelo PR @luisguilhermemsalmeida !!!
Perdão por toda demora pra revisar, foi erro total meu de me perder nas notificações do Github 😢
Fiz só uns ajustezinhos finais pra já deixar pronto pra mergear e publicar!
Muuito obrigado mano!! 🚀 ❤️
Se quiser trocar um papo mais sync fica a vontade de colar no Slack do BrasilAPI 😬
https://join.slack.com/t/brasilapi/shared_invite/zt-mkahmt0y-lfG~0VZL_LHcniodARNEwA
Adiciona parametro de timeout para serviços de cep
|
Valeu Luciano! |
Pessoal, adicionei timeouts para os serviços de CEP como parte do parâmetro de
optionsque é passado a função. Ontem e hoje (7 e 8 de Dezembro) os correios estão com problemas no sistema deles e a biblioteca está levando MUITO tempo para rejeitar a promise quando um cep inválido é enviado (pq espera os correios responder).Pontos de discussão principais:
proxyURLestava sendo passado para os providers, mas só encontrei esse código dentro dos arquivos da pasta/distInfinityValeu!