Liquidity error handling #516
No reviewers
Labels
No labels
blocked
breaking
bug
dependencies
duplicate
enhancement
good first issue
help wanted
question
tech debt
testing
wontfix
No milestone
No project
No assignees
1 participant
Due date
No due date set.
Dependencies
No dependencies set.
Reference
scion/arbitrader!516
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "liquidity-error-handling"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
In cases where
getLimitPrice()could not fetch the order book because of rate limiting (ie. 429 from the server) the resulting exception would not be caught properly and showed a message telling the user that the exchange did not have enough liquidity. This change improves the error handling so that we show more appropriate error messages when errors occur in this method.