feat: add BIP353 DNS payment instructions#236
Open
sdmg15 wants to merge 1 commit intobitcoindevkit:masterfrom
Open
feat: add BIP353 DNS payment instructions#236sdmg15 wants to merge 1 commit intobitcoindevkit:masterfrom
sdmg15 wants to merge 1 commit intobitcoindevkit:masterfrom
Conversation
ba3c713 to
a0f3e9d
Compare
Contributor
|
@sdmg15 Hi! Could you make it compile? |
49154f3 to
baf349b
Compare
Pull Request Test Coverage Report for Build 21818410493Details
💛 - Coveralls |
baf349b to
0e9756e
Compare
Author
|
@va-an Thanks for taking a look. I think it should be resolved now. |
0e9756e to
6724be5
Compare
Author
6724be5 to
20ef094
Compare
20ef094 to
977534a
Compare
977534a to
4852673
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds support for BIP353 payment instructions.
The following notable changes are done:
resolve_dns_recipientwhich receives a Human Readable Name (HRN) and returns the associated addresscreate_txfor it to support user specifying recipients asHRN.Notes to the reviewers
Here is a list of HRN that could be used to test the implementation:
bdk-cli -n bitcoin resolve_dns_recipient send.some@satsto.mebdk-cli -n bitcoin wallet -w regtest_default_wallet create_tx --to "bcrt1qe497k549sgw9trzym50tdlegq3xx4apjqynjqm:1000" --to_dns "send.some@satsto.me:5000"Changelog notice
resolve_dns_recipientto resolve the DNS payment instructions--to_dnstocreate_txcommand to allow sending to a Human Readable Name (HRN)Checklists
All Submissions:
cargo fmtandcargo clippybefore committingNew Features:
CHANGELOG.md