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

Enabling Keplr, support for multiple currencies when connecting Keplr wallet bugs #309

Open
avendauz opened this issue Jan 20, 2023 · 1 comment

Comments

@avendauz
Copy link

Encountered a few bugs when trying to connect a wallet via Keplr:

  • When trying to enable Keplr, an experimentalSuggestChain config is generated referencing the chain to be added to Keplr. The window allows a user to edit the config, but none of the changes persist when approving the chain in the popup in Keplr.

  • The experimentalSuggestChain config that is generated by ping-pub only supports one currency/asset, even if multiple assets exist on that chain (see explorer/src/chains/mainnet/stafihub.json, which has 4 different assets). Manual testing suggests that only the first asset of the array of assets of the json config are used in the experimentalSuggestChain config.

Screenshot 2023-01-20 at 12 23 14 PM

@liangping
Copy link
Member

Hey, I think you can edit the config json in the text box before enable.

"currencies": [
		{
			"coinDenom": "CRE",
			"coinMinimalDenom": "ucre",
			"coinDecimals": 6,
			"coinGeckoId": "crescent-network"
		},
        // you might need add those assets here.
	],

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

2 participants