Skip to content
This repository has been archived by the owner on Aug 22, 2018. It is now read-only.

Add 'Incomplete' to the getStats table #24

Open
lsaether opened this issue Jan 18, 2018 · 0 comments
Open

Add 'Incomplete' to the getStats table #24

lsaether opened this issue Jan 18, 2018 · 0 comments

Comments

@lsaether
Copy link
Member

Depending on the network conditions, a transaction may not get mined for a while and time out with the error:

Error: Transaction was not mined within 50 blocks, please make sure your transaction was properly send. Be aware that it might still be mined!

Since this is a soft error, it usually just means that the transaction will process successfully later. But this messes with the .getStats feature which uses the return receipt to track when a transaction was claimed or executed. We can add an 'Attempted' and/or 'Incomplete' column to the table to reflect that a transaction was sent but didn't complete successfully.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant