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.

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

Added addtransaction arg to payto/paytomany #1997

Merged
merged 4 commits into from
Sep 19, 2020
Merged

Added addtransaction arg to payto/paytomany #1997

merged 4 commits into from
Sep 19, 2020

Conversation

MrNaif2018
Copy link

This PR is a backport of spesmilo#6532
It makes it possible to use payto/paytomany+broadcast commands in concurrent environments when turning addtransaction flag on by adding tx to local history before a full network round-trip.

lib/wallet.py Outdated Show resolved Hide resolved
lib/wallet.py Outdated Show resolved Hide resolved
@cculianu
Copy link
Collaborator

Ok so you tested it? It's working for you now after the 4 changes?

@MrNaif2018
Copy link
Author

Yes, now with more changes I even got 50 payto in a row! Works! :D

@cculianu
Copy link
Collaborator

Ok, merging.

@cculianu cculianu merged commit 08b2e4b into Electron-Cash:master Sep 19, 2020
@MrNaif2018 MrNaif2018 deleted the payto-concurrency-improvements branch September 19, 2020 16:18
EchterAgo pushed a commit to EchterAgo/Electron-Cash that referenced this pull request Mar 9, 2021
* Added addtransaction arg to payto/paytomany
* Add to wallet history only if signed
* Check for duplicates in add_tx_to_history
* Fix add_tx_to_history duplicates check

---

This PR is a backport of spesmilo#6532
It makes it possible to use payto/paytomany+broadcast commands in concurrent environments when turning addtransaction flag on by adding tx to local history before a full network round-trip.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants