po-angular
po-angular copied to clipboard
feat(lookup): fazendo lookup cancelar requisiçoes ao fechar modal
< 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
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,