Skip to content

transaction handling

dju requested to merge txHandling into master

Feature Merge Request

Related Issue

closes #184 (closed)

Description of Feature

displaying a removable list of transactions below the button that triggered the respective transaction

Screenshots, Code, Links

Screenshot_from_2018-05-24_13-37-47

Related Tasks

  • [ ] make the txStatus component appear in the right place below the button (postponed for another mr)
  • display a link to etherscan in txStatus (that one would have to be adjusted to match the network)
  • have a different colored background (maybe like the way remix does it (see issue for a screenshot)
  • dispatch the transaction-related actions from stake, commit & reveal-buttons, addData
  • helper function sendAndReactToTransaction

Also, there is the somewhat odd behavior that the txStatus shows success, but the stake button is still active. Refreshing the page will then remove the txStatus-component and replace the button with the appropriate text. -> update the helper function can be used to trigger a re-fetching of the necessary information. As this MR is big enough as it is, I left this for !160 (closed)


Edited by dju

Merge request reports