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.

Sign up for GitHub

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

Jump to bottom

isMassaWalletEnabled fn naming is misleading #221

Open
pivilartisant opened this issue Apr 9, 2024 · 0 comments
Open

isMassaWalletEnabled fn naming is misleading #221

pivilartisant opened this issue Apr 9, 2024 · 0 comments

Comments

@pivilartisant
Copy link

The nomination of isMassaWalletEnabled(): Promise<boolean>; is misleading, as it checks if the Massa Wallet plugin is enabled in Massa Station. A dev can think instead that it checks if Massa Wallet is connected or enabled in their Dapp (it was my case).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pivilartisant