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 Keystone Wallet adapter #415

Merged
merged 1 commit into from
Jul 29, 2022
Merged

Conversation

zhaomengru2015
Copy link
Contributor

@zhaomengru2015 zhaomengru2015 commented May 10, 2022

What is this?

A wallet adapter for Keystone.


How to test?

  1. Run packages/starter/example project running
  2. On the example app, click Select Wallet under React UI
  3. Click Keystone
  4. Set Keystone Hardware Wallet to Solflare mode and select the address to sync
  5. Try actions

  • sign transactions
  • sign messages

@aaronisme
Copy link

@jordansexton would you please review our pr for adding the Keystone into the wallet adapter ?

@zhaomengru2015 zhaomengru2015 force-pushed the keystone branch 2 times, most recently from 16ec8f0 to 8a14bef Compare May 31, 2022 02:21
@jordaaash
Copy link
Collaborator

Hey, I want to address the lack of activity on your PR. Thank you for your patience!

Here's a followup on the state of wallet-adapter, the issues that individual adapters have in getting integrated with applications, and how we intend to improve things: #384 (comment)

Let's discuss in that thread to keep it in one place.

Copy link
Collaborator

@jordaaash jordaaash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Can you format with prettier so I can finish reviewing?

README.md Outdated Show resolved Hide resolved
packages/wallets/keystone/src/adapter.ts Outdated Show resolved Hide resolved
packages/wallets/keystone/src/adapter.ts Show resolved Hide resolved
@zhaomengru2015
Copy link
Contributor Author

@jordansexton Sorry for the delay. Thanks for reviewing and we've fixed the comments you mentioned above, please double check it.

@jordaaash jordaaash merged commit 41263f3 into anza-xyz:master Jul 29, 2022
@jordaaash
Copy link
Collaborator

Looks good, thanks!

@jordaaash
Copy link
Collaborator

Published @solana/[email protected], please test it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants