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

Improve Error Reporting #6

Closed
jansako opened this issue May 1, 2018 · 2 comments
Closed

Improve Error Reporting #6

jansako opened this issue May 1, 2018 · 2 comments

Comments

@jansako
Copy link

jansako commented May 1, 2018

When trying to execute a trade and only having one UTXO, OR trying to sell less than 10% of a UTXO (I can't tell which cause the message is unclear :-) ),
I get a 'something unexpected happened'.

image

Suggestion: message should be clear on next steps.

@lukechilds
Copy link
Owner

lukechilds commented May 1, 2018

Thanks for reporting @jansako

The error messages are provided to us directly from marketmaker. We agree they are not that clear and have made attempts to make some of them more obvious.

However unfortunately we don't get any error codes, only error messages, so it's very fragile for us to try and handle these manually. If the error message text from marketmaker ever gets changed in the future it will break all of our error handling code.

There are also some cases where we get not error code or error message, we just have missing properties in the response and have to guess what went wrong, like in the case with this error:

// TODO: Temp workaround for marketmaker issue
if (!result.pending) {
  this.setState({statusMessage: 'Something unexpected happened. Are you sure you have enough UTXO?'});
  return;
}

We need marketmaker to support error codes before we can reliably improve error handling but it is on our list to do.

More info in this issue: jl777/SuperNET#717

Please give it a thumbs up if it's something you'd like to see fixed.

@lukechilds lukechilds changed the title Unclear UTXO warning Better Error Reporting May 2, 2018
@lukechilds lukechilds changed the title Better Error Reporting Improve Error Reporting May 2, 2018
@sindresorhus
Copy link
Contributor

Moved to atomiclabs/hyperdex#280

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

No branches or pull requests

3 participants