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

Add API to request addresses #2371

Closed
kyranjamie opened this issue Apr 18, 2022 · 9 comments
Closed

Add API to request addresses #2371

kyranjamie opened this issue Apr 18, 2022 · 9 comments
Assignees
Labels
area:ledger area:request-accounts enhancement-p1 Critical functionality needed by many users, with no clear alternatives

Comments

@kyranjamie
Copy link
Collaborator

kyranjamie commented Apr 18, 2022

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

@markmhendrickson
Copy link
Collaborator

Related discussions: #2352 #1051 (comment)

@markmhendrickson
Copy link
Collaborator

markmhendrickson commented Apr 19, 2022

Note how the equivalent MetaMask UI appears to preselect just the currently / previously selected account in the user's session:

Screen Shot 2022-04-19 at 13 35 05

@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):

Screen Shot 2022-04-19 at 13 36 44

@landitus
Copy link

Thanks @markmhx. I'll include this scenario as I work on #2355

@landitus
Copy link

landitus commented Apr 20, 2022

Note how the equivalent MetaMask UI appears to preselect just the currently / previously selected account in the user's session:

  • Would the multiple account selection functionality just be available when the dApp requests the account details (this issue) and also when using the authentication methods?
  • Is selecting multiple accounts selection functionality a hardware wallet-specific feature? Why?

Trying to understand how the Connect wallet UX would evolve with this new method and if they should be treated as different UX patterns.

@markmhendrickson
Copy link
Collaborator

Would the multiple account selection functionality just be available when the dApp requests the account details (this issue) and also when using the authentication methods?

Just when the app requests accounts with the new API for requestAccounts or similar.

Is selecting multiple accounts selection functionality a hardware wallet-specific feature? Why?

Nope, multiple accounts can be selected regardless of whether the user is using a software or hardware wallet.

Trying to understand how the Connect wallet UX would evolve with this new method and if they should be treated as different UX patterns.

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 requestAccounts method or the old (and soon deprecated) authentication method.

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.

@landitus landitus removed their assignment Apr 22, 2022
@andresgalante andresgalante linked a pull request May 2, 2022 that will close this issue
6 tasks
@markmhendrickson markmhendrickson moved this from Development to Backlog in Hiro Wallet (DEPRECATED) May 16, 2022
@markmhendrickson markmhendrickson moved this from Backlog to Development in Hiro Wallet (DEPRECATED) May 16, 2022
@fbwoolf fbwoolf moved this from Development to Review in Hiro Wallet (DEPRECATED) Jun 20, 2022
@fbwoolf fbwoolf moved this from Review to Development in Hiro Wallet (DEPRECATED) Jun 20, 2022
@markmhendrickson markmhendrickson moved this from WIP to Assigned in Hiro Wallet (DEPRECATED) Jul 11, 2022
@markmhendrickson markmhendrickson moved this from Assigned to Backlog in Hiro Wallet (DEPRECATED) Aug 8, 2022
@markmhendrickson markmhendrickson moved this from Backlog to Assigned in Hiro Wallet (DEPRECATED) Sep 6, 2022
@kyranjamie kyranjamie moved this from Assigned to Backlog in Hiro Wallet (DEPRECATED) Oct 4, 2022
@svntax
Copy link

svntax commented Mar 12, 2023

Having something like eth_requestAccounts for stacks wallets would be very helpful for web games made in game engines like Unity and Godot. In many cases, reading the list of NFTs that an account owns is enough for a game to use those NFTs. I'm personally interested in adding Stacks wallet support in Godot, so I'd love to see this implemented.

@github-project-automation github-project-automation bot moved this to Enhancements backlog in Hiro Wallet (DEPRECATED) Mar 16, 2023
@markmhendrickson markmhendrickson moved this from Enhancements backlog to WIP in Hiro Wallet (DEPRECATED) Mar 16, 2023
@markmhendrickson
Copy link
Collaborator

@kyranjamie @mica000 putting this back on the board as WIP

@markmhendrickson markmhendrickson changed the title Add ability to request account details Add API to request addresses Mar 21, 2023
@markmhendrickson
Copy link
Collaborator

Now inclusive of taproot addresses: #3372

  • Stacks
  • Native SegWit
  • Taproot

@markmhendrickson
Copy link
Collaborator

Now supported via #3421

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:ledger area:request-accounts enhancement-p1 Critical functionality needed by many users, with no clear alternatives
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants