po-angular icon indicating copy to clipboard operation
po-angular copied to clipboard

feat(lookup): fazendo lookup cancelar requisiçoes ao fechar modal

Open LuanPonick opened this issue 1 year ago • 1 comments

< po-lookup >

DTHFUI-8000


PR Checklist [Revisor]

  • [ ] Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • [ ] Código (Boas práticas, nome de variavéis/métodos, etc.)
  • [ ] Testes unitários (Cobre a situação implementada e coverage está mantido)
  • [ ] Documentação (Clara, objetiva e com exemplos caso necessário)
  • [ ] Samples (A implementação possui exemplo no Labs/Caso de uso)
  • [ ] Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual? Na modal do lookup pode ser feita múltiplas requisições podendo gerar estranhezas caso não haja uma demora considerável no lookup, e no pior dos casos pode uma requisição sobrepor uma requisição.

Qual o novo comportamento? Caso o usuário tente fazer uma nova requisição ( dentro da modal do lookup ) e caso já tenha uma requisição pendente essa requisição pendente será cancelada e só logo após de ser cancelada será iniciada a nova e caso o usuário saia da modal serão canceladas as requisições pendentes.

Simulação Tela de carregamento sumindo sem a requisição ter terminado https://app.screencast.com/nyLgG1K2tP6Mp

LuanPonick avatar Nov 17 '23 16:11 LuanPonick

Boa tarde @LuanPonick , desculpe-nos pela demora em responder e ficamos muito felizes pela sua contribuição no projeto.

Testei localmente e a implementação atende ao que você precisa e agrega positivamente ao componente. Sua contribuição é muito importante por isso atualize sua branch com a última versão do po-lookup e também realize a cobertura de teste no método que você criou. Assim, daremos prioridade nesta PR para aprovação o mais breve possível.

Atenciosamente,

felipepetuco avatar Jan 22 '24 20:01 felipepetuco