-
Notifications
You must be signed in to change notification settings - Fork 146
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
Add API to request addresses #2371
Comments
Related discussions: #2352 #1051 (comment) |
Note how the equivalent MetaMask UI appears to preselect just the currently / previously selected account in the user's session: @landitus we'll need a similar modal for the wallet to support this issue's functionality Note that they have a second screen here that clarifies permissions, though I imagine we won't need one in our case since we can fit whatever guidance we need into just one screen (in which the user selects the accounts to share): |
Thanks @markmhx. I'll include this scenario as I work on #2355 |
Trying to understand how the Connect wallet UX would evolve with this new method and if they should be treated as different UX patterns. |
Just when the app requests accounts with the new API for
Nope, multiple accounts can be selected regardless of whether the user is using a software or hardware wallet.
I suspect that the app will show the same sort of "Connect wallet" option to users, regardless of whether they decide to trigger the new Both are ways to connect the wallet. They just differ in A) whether the account(s) are verified or not, and B) whether only a single account or multiple account values can be returned to the app as a result. |
Having something like |
@kyranjamie @mica000 putting this back on the board as WIP |
Now inclusive of taproot addresses: #3372
|
Now supported via #3421 |
Similar to Ethereum's
eth_requestAccounts
, the Hiro Wallet should provide a way to share account details that doesn't do auth, requiring signing of a JWT.Related MetaMask/metamask-extension#4703
The text was updated successfully, but these errors were encountered: