-
Notifications
You must be signed in to change notification settings - Fork 497
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
Remove pulsar.json #195
Remove pulsar.json #195
Conversation
`pulsar-2` is depreciated: `pulsar-3` info https://docs.scrt.network/secret-network-documentation/development/resources-api-contract-addresses/connecting-to-the-network/testnet-pulsar-3
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
the presence of pulsar breaks a the ability to suggest pulsar-3 if (pulsar-2 is defined here). Or the ability to suggest pulsar-2 (if pulsar-3 is defined here) https://github.com/chainapsis/keplr-wallet/blob/8ab00c015c12a274f7570f0aca7bb7c656f85230/packages/stores/src/core/interaction/chain-suggest.ts#L66
@luca992 How about keep the current file and change to |
As I was trying to say, people cannot suggest the |
@luca992 I got it! |
I would leave them if it were up to me. That way people can use them in their suggested configs. Also, once |
@luca992 I merge it. Thank you for contribution! |
The presence of pulsar breaks a the ability to suggest pulsar-3 if (pulsar-2 is defined here). Or the ability to suggest pulsar-2 (if pulsar-3 is defined here). Until pulsar-2 is taken down I don't think pulsar-2 or pulsar-3 should be defined here... unless a proper fix in keplr was implemented to make sure the chain ids actually match from the community repo to what the user supplied, and if not use what the person actually suggested.
https://github.com/chainapsis/keplr-wallet/blob/8ab00c015c12a274f7570f0aca7bb7c656f85230/packages/stores/src/core/interaction/chain-suggest.ts#L66