[FIX] Leitura do documento carregando mais de uma vez#15
Open
Marcos-Toledo wants to merge 1 commit intoclicksign:mainfrom
Open
[FIX] Leitura do documento carregando mais de uma vez#15Marcos-Toledo wants to merge 1 commit intoclicksign:mainfrom
Marcos-Toledo wants to merge 1 commit intoclicksign:mainfrom
Conversation
palloi
approved these changes
Feb 17, 2022
palloi
left a comment
There was a problem hiding this comment.
Boa! agora sim os eventos são removidos.
A ordem aqui faz diferença no build com o gulp.
|
Fala pessoal! tudo bem? Podem avaliar por favor ☝️ ? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Descrição
A função
unmountnã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 doaddEventlistenereremoveEventListener.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.addEventListenerdas linhas 27 e 30Print: build do código não alterado

Print: build do código alterado

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