multisender icon indicating copy to clipboard operation
multisender copied to clipboard

add last failed tx processing

Open dzhambulat opened this issue 6 years ago • 3 comments

this fix checks last failed tx when user enters token address

dzhambulat avatar Apr 12 '18 22:04 dzhambulat

so the issue was about to resend any failed tx by querying a tx hash

  1. Implement a new route to display input box to enter txhash
  2. verify if txhash is valid and it was sent to multisender contract address
  3. if it is, ask etherscan.io api for the reason of tx failure if it's out of gas, then
  4. parse tx input and move user to /3 screen where he can see all tx data like you do from screen 1 to 3
  5. send a transaction with higher gasLimit then the previous one.

rstormsf avatar Apr 12 '18 23:04 rstormsf

added page /#/checktx enter tx hash, and if it failed, user may push btton "Resend transaction", and it redirects to /#/3 and he can process it and change gas in metamask.

dzhambulat avatar Apr 15 '18 23:04 dzhambulat

I have done fixes for cleaning ( I forget to commit deleting of debug alerts and other fixes before last PR). I reviewed other classes, and I think that tokenStore and gasPriceStore should be refactored, because many methods violates The Single Responsibility Principle and it was hard to me make additional features and fixes.

dzhambulat avatar Apr 17 '18 15:04 dzhambulat