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

Harden wallet main UTXO lookup #726

Merged
merged 8 commits into from
Nov 6, 2023
Merged

Conversation

tomaszslabon
Copy link
Contributor

@tomaszslabon tomaszslabon commented Oct 31, 2023

#Refs: https://github.com/threshold-network/pm/issues/36.
This PR improves the wallet main UTXO lookup.
Before this PR we only looked for the wallet main UTXO in the last five transactions.
This PR changes this and now we look for the main wallet UTXO in the entire transaction history of the wallet.
We do it in an efficient way: first we retrieve the transaction hashes and then, starting from the most recent transaction, we check which transaction created the main UTXO.

@tomaszslabon tomaszslabon self-assigned this Oct 31, 2023
@tomaszslabon tomaszslabon added the 🔌 typescript TypeScript library label Oct 31, 2023
@tomaszslabon tomaszslabon force-pushed the harden-wallet-main-utxo-lookup branch from 1413bab to 02a3a69 Compare October 31, 2023 15:43
@lukasz-zimnoch lukasz-zimnoch added this to the typescript/v2.2.0 milestone Nov 1, 2023
@tomaszslabon tomaszslabon force-pushed the harden-wallet-main-utxo-lookup branch 4 times, most recently from 827ad7d to f65ef2a Compare November 1, 2023 15:59
@tomaszslabon tomaszslabon force-pushed the harden-wallet-main-utxo-lookup branch from f65ef2a to 6a1bffb Compare November 1, 2023 16:11
@tomaszslabon tomaszslabon marked this pull request as ready for review November 1, 2023 16:12
@lukasz-zimnoch lukasz-zimnoch merged commit 1d21c23 into main Nov 6, 2023
39 checks passed
@lukasz-zimnoch lukasz-zimnoch deleted the harden-wallet-main-utxo-lookup branch November 6, 2023 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔌 typescript TypeScript library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants