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

Suggest adding Default address column #37

Open
hosiawak opened this issue Aug 25, 2019 · 5 comments
Open

Suggest adding Default address column #37

hosiawak opened this issue Aug 25, 2019 · 5 comments
Labels
enhancement New feature or request

Comments

@hosiawak
Copy link
Contributor

In order to recover the user needs to try a few address types (if unknown).
It would be great to have all this info already available.
For starters we can use this matrix

I'll create a PR if @nvk OKs it.

@nvk
Copy link
Owner

nvk commented Aug 25, 2019

Many of these wallets will have more than one default no?

@nvk nvk added the enhancement New feature or request label Aug 25, 2019
@nvk
Copy link
Owner

nvk commented Aug 25, 2019

Maybe we should create another page called WALLET-DETAILS.MD and add an extra for each there. They would be longer and deep linked into each wallet section. Alternatively a doc file for each wallets/WALLETNAME.MD

@hosiawak
Copy link
Contributor Author

By default I mean whatever address type you get when you hit "Receive". It's usually just one type as seen on the above compatibility matrix. So for Trezor it's P2SH-P2WPKH but for Samourai P2WPKH and so on. Whatever type the user needs to select to have the greatest chance of recovery in Electrum (or similar software)

@hosiawak
Copy link
Contributor Author

I agree yet another column will make this table a bit less readable and packed though so maybe a details page for each wallet is a better option. I'll create a PR for another column and then you can decide.

@nvk
Copy link
Owner

nvk commented Aug 25, 2019

Uncertain how helpful that would be as wallets change their defaults as they progress. Maybe in this extended page/file we could have "Expected default addresses for this wallet:"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants