trading account.py uses the same critical error message for both Coinbase and coinbasepro.

whittlem / pycryptobot

Python Crypto Bot (PyCryptoBot)

Apache License 2.0

1.96k stars 739 forks source link

trading account.py uses the same critical error message for both Coinbase and coinbasepro. #804

Closed wattiii closed 1 year ago

wattiii commented 1 year ago

line 440 deals with coinbase, but it's critical error exception raised prompts users to an error with coinbase pro. It would be more clear if it said cionbase for the coinbase critical exception, and then said coinbase pro for the coinbase Pro error message.

whittlem commented 1 year ago

Thanks, changing it to:

raise Exception( f"TradingAccount: '{self.app.exchange}' API Error while getting balance." )