Skip to content

Conversation

@psgreco
Copy link
Contributor

@psgreco psgreco commented Feb 8, 2023

Fixes: #1208

@psgreco psgreco requested review from apoelstra and gwillen February 8, 2023 16:39
@gwillen
Copy link
Contributor

gwillen commented Feb 8, 2023

I am somewhat annoyed that this adds more special cases. We already have too many of those. It would be nice if that were avoidable.

HOWEVER, it's easy to verify that the special cases won't trigger in prod, at least, so they aren't risky from that angle.

utACK.

@jamesdorfman jamesdorfman merged commit a47f465 into ElementsProject:master Feb 13, 2023
@psgreco psgreco deleted the master-wsh-dynafed branch February 13, 2023 14:57
jamesdorfman added a commit to jamesdorfman/elements that referenced this pull request Mar 13, 2023
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.

Elements requires a dynafed transition before p2wsh pegins are enabled

3 participants