PR updates outdated references to the deprecated commerceblock/copay repository and replaces them with the actively maintained bitpay/wallet repository. The change ensures that dependencies and links point to the correct, maintained source.
BIP-175: fix links #1862
pull strmfos wants to merge 1 commits into bitcoin:master from strmfos:master changing 1 files +1 −1-
strmfos commented at 10:42 AM on May 30, 2025: contributor
-
Migrate references from deprecated commerceblock/copay to active bitpay/wallet 291ed0f006
- murchandamus changes_requested
-
murchandamus commented at 6:03 PM on May 30, 2025: contributor
Are you sure you copied the right link? As far as I can tell, the suggested link goes to a pull request that does not have anything to do with a reference implementation of BIP 175.
-
strmfos commented at 6:55 AM on May 31, 2025: contributor
https://github.com/BitLox/copay
Hey! This is a maintained fork of the original Copay wallet that supports the BIP-175 Pay to Contract Protocol implementation. The original commerceblock/copay repository is no longer available, but this fork preserves the required functionality for hierarchical deterministic (HD) wallet address generation with BIP32/BIP44 support.
Is this an appropriate replacement link, or do you have a better alternative you would recommend?
-
jonatack commented at 3:23 PM on June 1, 2025: member
As @murchandamus points out, the pull request link doesn't point to the code implementation. @strmfos Would you please update the link to one that does, or to the relevant section of the codebase, if self-contained? A repository that may contain the functionality somewhere is a bit too large.
-
strmfos commented at 9:37 AM on June 2, 2025: contributor
I found a specific implementation link for BIP-175: https://github.com/commerceblock/pay-to-contract-lib This repo contains the Hash to Partial Derivation Path Mapping function in contract.js file - exactly as described in the BIP-175 spec. Is this suitable as replacement reference?
-
jonatack commented at 5:39 PM on June 2, 2025: member
@strmfos That link looks redundant to the one in the next line; implementation-wise it only has that contract.js file.
This is a maintained fork of the original Copay wallet that supports the BIP-175 Pay to Contract Protocol
I might be missing something, but that repo references several BIPs, yet I didn't see a reference there to BIP175.
As BIP175's status is Rejected, perhaps remove line 247 containing the broken link?
-
strmfos commented at 5:46 PM on June 2, 2025: contributor
I really reviewed a lot to find a proper replacement for the link, but I’ve shared everything I found.
So it might be reasonable to remove line 247 for now—if that works for you, I’ll go ahead and do it.
-
- jonatack closed this on Jun 2, 2025
Contributors