Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Apr 30, 2021

+ Bitcoin Core only supports valid bitcoin `address` for URI
+ Only first example will work from BIP 21 and others will give error
@ghost
Copy link
Author

ghost commented Apr 30, 2021

PR #176 mentioned for BIP 21 has no commits so maybe we can also add/replace it with 70f5535

@DrahtBot DrahtBot added the Docs label Apr 30, 2021
* [`BIP 14`](https://github.com/bitcoin/bips/blob/master/bip-0014.mediawiki): The subversion string is being used as User Agent since **v0.6.0** ([PR #669](https://github.com/bitcoin/bitcoin/pull/669)).
* [`BIP 16`](https://github.com/bitcoin/bips/blob/master/bip-0016.mediawiki): The pay-to-script-hash evaluation rules have been implemented since **v0.6.0**, and took effect on *April 1st 2012* ([PR #748](https://github.com/bitcoin/bitcoin/pull/748)).
* [`BIP 21`](https://github.com/bitcoin/bips/blob/master/bip-0021.mediawiki): The URI format for Bitcoin payments has been implemented since **v0.6.0** ([PR #176](https://github.com/bitcoin/bitcoin/pull/176)).
* [`BIP 21`](https://github.com/bitcoin/bips/blob/master/bip-0021.mediawiki): The URI format for Bitcoin payments has been implemented since **v0.6.0** ([PR #176](https://github.com/bitcoin/bitcoin/pull/176)). Only address is supported in URI as mentioned in first [example](https://github.com/bitcoin/bips/blob/master/bip-0021.mediawiki#examples).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: "Only [the] address is supported in [the] URI as mentioned in [the] first example"

Copy link
Author

@ghost ghost May 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"[the] address" or "[an] address" or "[a] Bitcoin address"?

@Rspigler
Copy link
Contributor

Rspigler commented May 2, 2021

I tested here: bitcoin-core/gui#280 (comment) and a label was still present for a URI with a valid address

@ghost
Copy link
Author

ghost commented May 2, 2021

I tested here: bitcoin-core/gui#280 (comment) and a label was still present for a URI with a valid address

I just tried 4 examples in v0.21.1 and they are working. Will close this PR and respond in bitcoin-core/gui#280 after testing.

@ghost ghost closed this May 2, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants