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

RFC 0008: Standard JavaScript Mina Wallet Provider API #13

Merged
merged 4 commits into from
May 17, 2024

Conversation

teddyjfpender
Copy link
Contributor

@teddyjfpender teddyjfpender commented Jan 15, 2024

🚀 Introduction to the Innovation

Here is RFC-0008! It's about introducing a standard JavaScript Mina Wallet Provider API to the ecosystem.

🤔 Why This Is Important

There exists no standard way for wallets and zkApps to interact. Standardising this API means zkApps have to spend less time integrating against many wallets, allowing any wallet to interact with any zkApp.

💭 Seeking Your Input

Your expertise, feedback, and perspective are needed to ensure that all requirements for a standardised wallet provider and any implementation are met!

@NorbertBodziony
Copy link

IMO we should also consider autodetection of wallets in this RFC. API of networks like Solana are based on https://github.com/wallet-standard/wallet-standard. This skips a lot of issues related to supply chain attack. ETH recently also released EIP addressing this issue https://eips.ethereum.org/EIPS/eip-6963.

Happy to make MVP after wallet interface gets finalized.

kantp
kantp previously approved these changes May 17, 2024
Copy link
Contributor

@kantp kantp left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for all the comments and suggestions @lvshaoping007!

chad11111
chad11111 previously approved these changes May 17, 2024
@kantp kantp dismissed stale reviews from chad11111 and themself via d77e2f9 May 17, 2024 09:59
@kantp kantp requested a review from halsaphi May 17, 2024 10:21
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.

6 participants