search

lousando / briskett

A delicious Tezos wallet interface for Trezor.
https://briskett.app
MIT License
5 stars 8 forks source link

Briskett addresses #5

Closed Tarskin closed 2 years ago

Tarskin commented 2 years ago

Briskett lists only 5 addresses, with the derivation paths ending on /0 through to /4. However, as many users will be migrating here from SimpleStaking there is a need to be able to access the same addresses that could be accessed through there. They also added the /0/0 and /0/1 paths (for address 6 and 7). Therefore, it would be a nice feature to add this to Briskett (or the option to enter a path manually?).

Alternatively, instead of selecting the addresses before the entire process, how about showing the addresses, with balance after retrieving the public keys from the trezor and allowing the user to select the address from the shown list?

lousando commented 2 years ago

Wouldn't be too difficult to add the other addresses SimpleStaking supports.

To pull all addresses from the Trezor would be a bit more re-work, but it's do-able. The UI would just need to be tweaked a bit to have some sort of address switcher.

Tarskin commented 2 years ago

Any objection if I draft a PR for this?

lousando commented 2 years ago

No, no objections. Any help is appreciated :)

lousando commented 2 years ago

Missing Briskett addresses 6 & & added with #6

Closing this and will add create a new ticket for the dynamic address changing.