-
Notifications
You must be signed in to change notification settings - Fork 57
Exception: KeyError: 'sends'. when calling asset BTC. #91
Comments
Do you want submit a Pull Request for this to |
Can we close this now? |
I just submitted the issue because I thought the maintainers might like to be aware and do something about it. Seems to me that a well designed software handling people's money should handle all user input as gracefully as possible. If looking up BTC is not supported, then it should probably give a friendly error message in that case. |
Absolutely - thanks for taking time to let us know and share the patch, we appreciate it! We'll try to improve console.py shortly, maybe by adding improvements to another (or new) command. |
I'm not going to submit a pull request. I would if I'd done something significant, but that was just a one-liner that quickly resolved the issue for me, so I included it as example -- likely not the correct fix. of course feel free to use it if you like it. |
For future reference:
(I wasn't sure whether the workaround was that with it the API returns a "not supported" message, or it actually treats BTC as another CP asset name. Interestingly the supply amount is correct too).
|
Guess the correct solution is to return a "not an asset" error (BTC is not an asset anyways) |
Got this error.
I fixed (or worked around) it with:
The text was updated successfully, but these errors were encountered: