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

[FIX] Leitura do documento carregando mais de uma vez #15

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

Conversation

Marcos-Toledo
Copy link

Descrição

A função unmount não está removendo os eventos corretamente.

Olhando o arquivo compilado o evento setado para o window message não é o mesmo ao realizar o unmount().

Para a correção alteramos a ordem de declaração da função handle(), para que a referencia na hora da compilação seja a mesma para adição do addEventlistener e removeEventListener.

O problema foi observado dentro de um fluxo de steps, onde era carregado os eventos do clicksign em um dos steps, se o usuário precisava voltar para um step anterior ao do clicksign por algum motivo, e depois quisesse seguir em frente com os steps, os eventos era carregado mais de uma vez, e com isso no momento de assinar o documento era feito múltiplas requisições.

Issue tracker

[Link para o card no Kanbanize.]

Screenshots (para mudanças de UI, se houver)

Observar o window.addEventListener das linhas 27 e 30

Print: build do código não alterado
image

Print: build do código alterado
image

Links e observações

[Links úteis que podem contextualizar e ajudar o revisor, por exemplo para a página de uma dependência que escolheu adicionar, ou um código que se inspirou, ou documentação externa (docs de uma API, do Vue, do Rails, etc).]

Checklist para poder mergear

  • O código do PR inclui (ou já possui) testes para o código nele
  • Os checks de linters estão passando
  • Os checks de testes estão passando

Copy link

@palloi palloi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa! agora sim os eventos são removidos.

A ordem aqui faz diferença no build com o gulp.

@palloi
Copy link

palloi commented Feb 18, 2022

@dlibanori @mbbernstein

Fala pessoal! tudo bem? Podem avaliar por favor ☝️ ?

@eaebob eaebob changed the title [FIX] - Leitura do documento carregando mais de uma vez [FIX] Leitura do documento carregando mais de uma vez Jan 19, 2024
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