Closed felipepimsil closed 12 months ago
Eu concordo com a sugestão. É mais seguro esperar uma confirmação propriamente dita do que simplesmente um 'setTimeout' do prazo estimado de se criar um novo bloco. Porém, são apenas exemplos e que cumprem o seu papel como tal. Cabe ao participante implementar a melhor abordagem em seu próprio contexto.
Uma sugestão à parte é evitar editar no código o que não tem relação com a necessidade da PR. Você alterou várias linhas apenas trocando ' por " e também quebrando linhas. Isso deixa difícil de se analizar o que de fato foi a alteração, sem falar que não é boa prática editar um código público apenas para se encaixar em algum gosto pessoal.
@thiagodeev Sim, em relação à funcionalidade do exemplo não alterará nada. Na realidade, para testes locais, as transações são meio que instantâneas, então só remover o delay
já ajudaria. Só fiz a alteração porque no caso de alguém querer "aprender", verá algo que não está 100% correto.
E quanto a sugestão, bem lembrado. Acabei que, ao salvar, formatar todo o arquivo e fazendo essas alterações.
Prezado @felipepimsil,
Por ser uma melhoria nos códigos de exemplos, com um baixo valor agregado, a equipe de desenvolvimento optou por rejeitar sua PR.
Att Aldenio.
Olá me liga preciso falar com vc 37991823326zap Firmino
Não acho válido dizer que deve-se esperar 1 bloco, pois nem sempre a transação é mintada em 1 bloco após o envio. O jeito certo é dizer que deve esperar que a transação seja confirmada e seja válida
status === 1
. Assim teremos a mudança de estado desejada.Caso a transação falhe ou não exista, as transações seguintes que dependem do estado gerado por
enableAccount
irão falhar.