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

Removed link and reference to SLIP-0173 #1481

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

Conversation

dplusplus1024
Copy link
Contributor

SLIP-0173 is not relevant to BIP173 due to not containing any additional information about bitcoin.

SLIP-0173 is not relevant to BIP173 due to not containing any additional information about bitcoin.

Co-authored-by: Reese Russell <git@qrsnap.io>
@apoelstra
Copy link
Contributor

It may be irrelevant to Bitcoin, but it redirects a common request about BIP-173, without needing any action from the BIP maintainer or others here.

@clarkmoody
Copy link
Contributor

clarkmoody commented Jul 24, 2023 via email

@dplusplus1024
Copy link
Contributor Author

dplusplus1024 commented Jul 24, 2023

@apoelstra @clarkmoody Thank you both for responding. It is my perspective that linking these projects may be perceived as an official endorsement for them. However, I can understand the desire to stop needless PRs. The question is, what is more important - preventing PRs or preventing users from being led astray?

@apoelstra
Copy link
Contributor

I don't think it'd be perceived as an "official endorsement". Perhaps an "official acknowledgement", but that's fairly accurate -- if we were to add new HRPs in Bitcoin for something, we would check SLIP-173 to avoid colliding, because it's an easy check and it's good social grace.

@murchandamus
Copy link
Contributor

It sounds to me that the BIP authors declined this proposed change to their BIP.

@murchandamus murchandamus reopened this May 1, 2024
@murchandamus
Copy link
Contributor

Oh oops, I got the impression that Andrew and Clark were the authors, but just realized that BIP173 is bech32 of course. Paging @sipa and @gmaxwell.

@murchandamus murchandamus added Proposed BIP modification Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified labels May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified Proposed BIP modification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants