-
Notifications
You must be signed in to change notification settings - Fork 42
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
synchronize BIP 77 with code #458
Comments
I started pushing these changes to a secondary branch so I don't interrupt the reviewers until we're ready for a review. FYI Note that Short IDs likely also need to be uppercase since they're parsed as having a case-sensitive bech32 HRP in our code, too |
Yes, with the same rationale as fragment parameter, although the |
77 links to the draft RFC. What more would make it sufficiently unambiguous?
|
Pushed changes to the secondary branch |
The BIP 77 spec is not fully up to date with these changes:
Additionally the document can be improved:
The text was updated successfully, but these errors were encountered: