[bisq-network/bisq] Add API 'gettrades' method (PR #5976)
Christoph Atteneder
notifications at github.com
Thu Jan 27 16:58:41 CET 2022
@ripcurlx commented on this pull request.
> @@ -292,7 +298,10 @@ Trade getTrade(String tradeId) {
List<TradeModel> getOpenTrades() {
coreWalletsService.verifyWalletsAreAvailable();
coreWalletsService.verifyEncryptedWalletIsUnlocked();
- return tradeManager.getTrades().stream().map(t -> (TradeModel) t).collect(Collectors.toList());
+ return tradeManager.getTrades().stream()
+ .map(t -> (TradeModel) t)
I don't mean so much about the return types of the method, but rather the fact, that in this case it is unnecessary to cast.
--
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5976#discussion_r793755581
You are receiving this because you are subscribed to this thread.
Message ID: <bisq-network/bisq/pull/5976/review/865082759 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220127/4c767a64/attachment.htm>
More information about the bisq-github
mailing list