wisespace-io / binance-rs

Rust Library for the Binance API
Other
671 stars 300 forks source link

Change types in Kline from i32 to i64 #139

Closed siegfried closed 2 years ago

siegfried commented 2 years ago

number_of_trades in KlineSummary is i64 while in Kline is i32. @wisespace-io

siegfried commented 2 years ago

By the way, shall we match the field names in Kline and KlineSummary, too? @wisespace-io

wisespace-io commented 2 years ago

By the way, shall we match the field names in Kline and KlineSummary, too? @wisespace-io

Yes, I think it makes sense.

siegfried commented 2 years ago

@wisespace-io Please review.

siegfried commented 2 years ago

I've renamed them.