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

Slightly misleading error message (alternative included) #770

Open
mikemaccana opened this issue Apr 19, 2023 · 0 comments
Open

Slightly misleading error message (alternative included) #770

mikemaccana opened this issue Apr 19, 2023 · 0 comments

Comments

@mikemaccana
Copy link
Contributor

mikemaccana commented Apr 19, 2023

Describe the bug

This error implies that wallet adapters must be installed for all wallets supported by the app:

Your application is presently configured to use the UnsafeBurnerWalletAdapter. Find and remove it, then replace it with a
list of adapters for wallets you would like your application to support. See https://github.com/solana-labs/wallet-
adapter#usage for an example.

A possible alternative:

Your application is presently configured to use the UnsafeBurnerWalletAdapter. Find and remove it. By default wallet-
adapter supports all wallet-standard wallets, you may also wish to add specific adapters for wallets you would like your
application to support. See https://github.com/solana-labs/wallet-adapter#usage for an example.

To Reproduce
Start the packages/starter/example project.

Expected behavior
An accurate message.

Desktop (please complete the following information):

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:  Ubuntu 22.04.1 LTS
Release:  22.04
Codename: jammy

Additional context
Would send a PR but thought we'd want to discuss and tweak it a little.

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

No branches or pull requests

1 participant