[bisq-network/bisq] Add API 'gettrades' method (PR #5976)

Christoph Atteneder notifications at github.com
Thu Jan 27 09:50:27 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)

Why do you need to map it here?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5976#pullrequestreview-864545327
You are receiving this because you are subscribed to this thread.

Message ID: <bisq-network/bisq/pull/5976/review/864545327 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220127/303c0441/attachment.htm>


More information about the bisq-github mailing list