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

decred: Add input_derive_script. #2703

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JoeGruffins
Copy link
Contributor

@JoeGruffins JoeGruffins commented Dec 27, 2022

Decred shares code with bitcoin and a recent change broke signing stake
based inputs. Add decred specific script construction and fix tests.

This fixes a bug introduced by c9e6ca3

This is partly my fault as the test input was flawed and did not have the proper script opcode prefix.

Feel free to take these changes and adjust as you like. I do not need credit. Just hope to get the bug fixed asap.

Decred shares code with bitcoin and a recent change broke signing stake
based inputs. Add decred specific script construction and fix tests.
@JoeGruffins
Copy link
Contributor Author

@matejcik Hey buddy. Any chance you could have a look at this. Almost a year old. Thanks.

@teknico
Copy link
Contributor

teknico commented Oct 3, 2023

IIUC, this change is needed to stake Decred coins. This is the remaining blocker for me and other Decred holders. We could finally use our Model T’s for Decred, after all these years.

@peterzen
Copy link
Contributor

peterzen commented Nov 5, 2023

@matejcik please merge this.

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

Successfully merging this pull request may close these issues.

None yet

3 participants