Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Resolve issues found in reviewed PRs 4699, 4703, 4711 [, ...] #4731

Merged
merged 20 commits into from Nov 2, 2020
Merged

Resolve issues found in reviewed PRs 4699, 4703, 4711 [, ...] #4731

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
a3631a0
Implement api methods 'keepfunds', 'withdrawfunds'
ghubstan Oct 26, 2020
f1db254
Make formatSatoshis visible for testing
ghubstan Oct 27, 2020
b8ae566
Add method for printing current jupiter test name
ghubstan Oct 27, 2020
1e25be5
Test trade closing api methods 'keepfunds' withdrawfunds'
ghubstan Oct 27, 2020
2746b27
Fix apitest dummy payment acct init bug
ghubstan Oct 27, 2020
027a7d5
Stub out canceloffer api method
ghubstan Oct 27, 2020
0f1d4f8
Fix typo
ghubstan Oct 27, 2020
b38507c
Implement api method 'canceloffer'
ghubstan Oct 27, 2020
91a2e2c
Add canceloffer test
ghubstan Oct 27, 2020
8e51e1e
Upgrade jupiterVersion = '5.7.0'
ghubstan Oct 28, 2020
d6524bf
Improve apitest cmd line console logging
ghubstan Oct 28, 2020
2a05203
Remove dead code
ghubstan Oct 28, 2020
f61f148
Refactor api test fixture setup
ghubstan Oct 28, 2020
e8e55d2
Remove redundant ScenarioTest superclass
ghubstan Oct 28, 2020
c2f8db3
Reduce apitest case execution time
ghubstan Oct 28, 2020
0656c57
Add null checks in PendingTradesViewModel
ghubstan Oct 30, 2020
f764e9f
Simplify TradeUtil#getPaymentMethodNameWithCountryCode
ghubstan Oct 30, 2020
3cd3bf0
Fix grammar / typo
ghubstan Oct 30, 2020
c27d5a6
Use Bisq's timer in API's 'unlockwallet timeout(s)'
ghubstan Oct 31, 2020
fcdfc68
Verify closedTradableManager.getTradableById return value is a Trade …
ghubstan Oct 31, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 3 additions & 1 deletion core/src/main/java/bisq/core/api/CoreTradesService.java
View file
Open in desktop
Expand Up @@ -21,6 +21,7 @@
import bisq.core.btc.wallet.BtcWalletService;
import bisq.core.offer.Offer;
import bisq.core.offer.takeoffer.TakeOfferModel;
import bisq.core.trade.Tradable;
import bisq.core.trade.Trade;
import bisq.core.trade.TradeManager;
import bisq.core.trade.TradeUtil;
Expand Down Expand Up @@ -199,7 +200,8 @@ private Optional<Trade> getOpenTrade(String tradeId) {
}

private Optional<Trade> getClosedTrade(String tradeId) {
return closedTradableManager.getTradableById(tradeId).map(value -> (Trade) value);
Optional<Tradable> tradable = closedTradableManager.getTradableById(tradeId);
return tradable.filter((t) -> t instanceof Trade).map(value -> (Trade) value);
}

private boolean isFollowingBuyerProtocol(Trade trade) {
Expand Down