6bd101ebb3 Remove link to old non segwit version and change new version link to use JoinMarket github organization (Kristaps Kaupe)
e8ec595d31 Shebang and +x for ob-watcher.py (Kristaps Kaupe)
80333b2872 Make ob-watcher independent of current working directory (Kristaps Kaupe)
8997147848 Terminate ob-watcher immediately if HTTP server fails to start (Kristaps Kaupe)
Pull request description:
* Terminate immediately if HTTP server fails to start.
* Make `ob-watcher.py` runnable from any directory.
* Shebang and +x (like was done in #524 for other scripts).
Top commit has no ACKs.
Tree-SHA512: 92b622fc9ad7cc910c1fe354146287998348e26a94909d52961ab50c5c39e2db9707341c2ce5e1943e1b1d4d49a4ecc86cafdf395a88346a2f43f0ad0bc2283d
f2e100e fix sample schedule for testnet (Adam Gibson)
031b8f6 Update tumbler doc for the new schedule format (chris-belcher)
Tree-SHA512: c456863cfe4e18a81e4e769a6c28ef1b01c2a516332281cb7868cb2c7e7e9f62e0c566185911958803ca3c22ae03b764b4c2b3ac969fd47614d728cca15b88f3
6fa7f4cdbb Display address for UTXO on wallet-tool.py freeze (Kristaps Kaupe)
Pull request description:
Suggested by @chris-belcher in #575. Also changes value output from `Y sats` to `X BTC (Y sat)` as in other places.
Top commit has no ACKs.
Tree-SHA512: 60747466f08b282edc9e3dacc6e038de82905e6364065c4b51a084a49832b0f430f92553bf6339db80005db55ca688b863a360e88cd99a6dbf3d5154344b09c3
As per discussion in #569 the use of options.txfee
to set fee rates conflicted with its earlier use
to make a first guess of fees for user warnings.
Now this connection is removed, the guess for warning
calculation is done independent of this option setting.
a539345be3 Implement BIP21 in GUI (Kristaps Kaupe)
Pull request description:
Allows pasting BIP21 URI in address field for single sends. See https://github.com/JoinMarket-Org/joinmarket-clientserver/pull/560#issuecomment-622920476. Also added stripping address field input of extra whitespace at beginning / end. <del>Change with `raise ValueError` in `decode_bip21_uri()` is needed because `amount_str` can be invalid and that raised `decimal.InvalidOperation` on `amount_to_sat()` call.</del><ins>(UPDATE - that one is already merged in with #568)</ins>
Top commit has no ACKs.
Tree-SHA512: 079ca866cc0cded1624531ad6397fcdeeb7ebed7c6eab778e8640561387a3c08e39be2dc137df785f3aae2ad1b4a56b636d1f304ef854da27cf651293d78266f
Prior to this commit, if a tumbler coinjoin negotiation
failed in Phase 2, then the retry as per the logic in
taker_utils.tumbler_taker_finished_update would always
attempt to retry the transaction with those counterparties
that returned valid !sig responses. However this ignored
the case that all the counterparties responded validly,
but there was a mempool conflict in the created transaction.
After this commit, if it is detected that all counterparties
responded, it is assumed that a mempool conflict or similar
occurred with the transaction, and therefore it is better
to fallback to a schedule tweak and choose randomly again,
not to fix the counterparty set (which is likely to result
in failing again).
Probably a value from the past, when BTC was worth considerably less.
To align with yg-basic, which already uses this lower min size.
Improves orderbook liquidity for smaller amounts.
34fe703 Freeze/unfreeze all selected UTXO's from context menu (Kristaps Kaupe)
Tree-SHA512: 5737ae7f2bfd11a6dda21a69d878ae17a200562f73486948c4ff85f81b1f7e2f3131060041f55a580daa8118633490af69fdb4765356e1cb8b3de27a8543201c
49ea4e44bf Correct display of wallet mnemonic extension in cli (Kristaps Kaupe)
6f90e30424 Fix "Show seed" in Qt for wallets with mnemonic extension (Kristaps Kaupe)
Pull request description:
Fixes error:
```
Traceback (most recent call last):
File "joinmarket-qt.py", line 1725, in showSeedDialog
self.displayWords(*self.wallet_service.get_mnemonic_words())
File "joinmarket-qt.py", line 1776, in displayWords
text += "<br/><br/>Seed extension: <strong>" + mnemonic_extension + "</strong>"
TypeError: must be str, not bytes
```
Top commit has no ACKs.
Tree-SHA512: 0484ce53e746512c159399d191e350bf79168d294e8b614ae9178809976c87e7cc227187fa312fc2ca3c78e4befc04826cd4d82833627dc19f97ea26c0ee7865
faaf51eada Add BIP21 bitcoin payment URI support to sendpayment.py (Kristaps Kaupe)
Pull request description:
In future could and should be improved to add this also to Qt GUI, but didn't want to do that now, need to figure out proper UX there.
This is needed for #557.
Top commit has no ACKs.
Tree-SHA512: a3911f62c27b71796b27fb74be09f2cfb8e2f6f5b0ef0909ecafb16ef07b8023ea0f193f829bba2e580dc0880016d7c1e051e8d25f324e4a1178d112a69fa16c
Prior to this commit, the callback method in the
DaemonServerProtocol, on_commitment_seen, was using
a reference to jm_single (to access the config var
accept_commitment_broadcasts) which was not valid
as jm_single() is part of jmclient and is not accessible
to jmdaemon. This error was being swallowed by a finally:
block in the message channel method check_for_commitments,
resulting in public broadcast hp2 messages being ignored.
This commit removes the reference to accept_commitment_broadcasts,
thus resulting in all publically broadcast hp2 messages being
stored in the commitmentlist.
Kristaps Kaupe
Adam Gibson
chris-belcher
AlexCato
nixbitcoin
undeath