Add stratos pubkey support #3896
Conversation
|
@Xiong-stratos is attempting to deploy a commit to the OsmoLabs Team on Vercel. A member of the Team first needs to authorize it. |
| @@ -566,6 +566,10 @@ export class CosmosAccountImpl { | |||
| return "/injective.crypto.v1beta1.ethsecp256k1.PubKey"; | |||
There was a problem hiding this comment.
Thank you for opening this PR. Is this code addressing a specific issue within the app? I believe we no longer use this store for signing transactions. If so, could you please point it out so we can ensure proper QA? Also, we already include Stratos in our getPublicKeyTypeUrl which is used in all of our signing methods.
There was a problem hiding this comment.
@JoseRFelix we have this issue during integration osmosis-labs/assetlists#1946 (comment) . We have resolved this in keplr wallet, but on ui looks like it use own logic and broadcast without keplr (so keplr looks like only to sign).
There was a problem hiding this comment.
Do you know if this is working locally?
There was a problem hiding this comment.
@JoseRFelix tested, still same problem, looks like codechanges not fixing an issue and problem in som another place.
What is the purpose of the change:
Add stratos pubkey support
Linear Task
Brief Changelog
Testing and Verifying