search

leapwallet / cosmos-metamask-snap

Leap Metamask Cosmos Snap for making cosmos transactions
https://leapwallet.io
Other
8 stars 7 forks source link

Update shentu bech32 prefix to shentu #79

Open HS-Joe opened 7 months ago

HS-Joe commented 7 months ago

Shentu prefix was updated in version 2.8.0, see https://github.com/shentufoundation/shentu/releases/ Version 2.8.0 changes the Bech32 address prefix to 'shentu'.

Using the certik prefix causes issues with rest endpoints, for example https://shentu-rest.publicnode.com/cosmos/staking/v1beta1/delegations/certik1npauvhjz4zm4pejsquh3ey3lh3ec207k7p978e?pagination.limit=1000

Updating bech32PrefixAccAddr should be sufficient as the value is passed to packages/snap/src/wallet/wallet.ts#create method in addressPrefix which will generate the correct public address.

HS-Joe commented 5 months ago

Hey @baryon2 could you check this PR? I know that Shentu is not a major chain but the change is minor. Thanks!