Fix empty ccxt_order['trades'] error

Dave-Vallance / bt-ccxt-store

Fork of Ed Bartosh's CCXT Store Work

MIT License

423 stars 187 forks source link

Fix empty ccxt_order['trades'] error #47

Closed qinwang-ai closed 3 years ago

qinwang-ai commented 3 years ago

Fix empty ccxt_order['trades'] error

Dave-Vallance commented 3 years ago

Thanks - I don't have time to look at it but since @rvleij approved it, will merge in.

rvleij commented 3 years ago

Oh haha, I didn't expect that when I approved it. but I'm running this fix now on Bybit (lots of issues / trial and error, but unrelated) and on Binance, so it's ok. I won't be popular if I break everyones installation though :)