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

Respect locked utxos #2563

Closed
wants to merge 10 commits into from
Closed

Conversation

dannydeezy
Copy link

Locked utxos should not be available to spend. (#2496)

This PR attempts to ensure that if a utxo has been locked with the lockunspent bitcoin RPC call, then it will not be selected as a cardinal utxo. This enables people to safely lock utxos that they don't want ord to spend when doing inscriptions or sends (unless they are deliberately inscribing/sending that particular utxo)

@dannydeezy
Copy link
Author

@raphjaph new version of (#2520) here

@dannydeezy dannydeezy mentioned this pull request Oct 19, 2023
@raphjaph
Copy link
Collaborator

@dannydeezy we did a huge refactor of the inscribe logic, there's a bunch of conflicts now in the branch. If you could resolve them I can review. The new batch code is helpful to look at though, could be interesting for your service :)

@rot13maxi
Copy link
Contributor

hey @dannydeezy I lifted your changes from here and did a tiny refactor on top of the batch interface here: #2618. Happy to PR against your branch if you want, or we can roll any changes you have into that PR. Either works for me.

@raphjaph
Copy link
Collaborator

raphjaph commented Nov 8, 2023

Closing this since all the work is happening in #2618

@raphjaph raphjaph closed this Nov 8, 2023
@dannydeezy
Copy link
Author

hey @dannydeezy I lifted your changes from here and did a tiny refactor on top of the batch interface here: #2618. Happy to PR against your branch if you want, or we can roll any changes you have into that PR. Either works for me.

awesome thanks @rot13maxi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants