Closed TSavo closed 4 years ago
@badgerwithagun Ditched the expanded API in favor of Poloniex having one shared observable. :)
@badgerwithagun Great call on the memoizer. This has been rebased to develop and the memoization is now present and tested.
@badgerwithagun All set. Memoized in it's entirety (not by method reference to share), final, and dependency versions removed.
To better support #124 and #98, PoloniexStreamingMarketDataService has been changed to have a single subscription for Ticker which is then shared by all subscribers to the Ticker.
To get all the Tickers in a single observable, it is suggested the client create a list of all currencyPairs on the exchange, and use something like the following:
Observable<Ticker> allTickers = Observable.fromIterable(currencyPairs).flatMap(pair -> streamingMarketDataService.getTicker(pair));