Allow TickerMessage to have null dailyChange?

bit-oasis / bitfinex-api

0 stars 4 forks source link

Allow TickerMessage to have null dailyChange? #4

Open foowie opened 6 years ago

foowie commented 6 years ago

https://github.com/bit-oasis/bitfinex-api/blob/master/src/BitOasis/Bitfinex/Websocket/Channel/Ticker/TickerChannel.php#L68 - 3rd param might be null?

davefu commented 6 years ago

I think not (at least within BFX), if you point to Ticker::dailyChange entity (DB column), it's nullable only because of Bitstamp, cause there wasn't such value in their API...