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

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

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

Conversation

LuanPonick
Copy link

@LuanPonick LuanPonick commented Nov 17, 2023

< 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

@felipepetuco
Copy link
Contributor

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 felipepetuco added the status: awaiting author Additional information has been requested from the author label Jan 22, 2024
@alinelariguet alinelariguet removed the status: awaiting author Additional information has been requested from the author label Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants