harmony-one / chrome-extension-wallet

Harmony Chrome Extension Wallet
https://chrome.google.com/webstore/detail/harmony-one-wallet/fnnegphlobjdpkhecapkijjdkgcjhkib
60 stars 36 forks source link

Invalid hexidecimal string #94

Closed yelllowsin closed 3 years ago

yelllowsin commented 3 years ago

I am having an issue using One Wallet 1.1.1 when trying to approve a proposal via Snapshot here: https://hmny-governance.netlify.app/#/staking-testnet/proposal/QmZrKwfYV6xXb52xupHRpH52tywANJBhrPBPUtJjGDxgkL

As you can see, this proposal is already approved now because when i tried via Metamask it worked.

Detailed error message here: Signing by Test 1 one170xqsfzm4xdmuyax54t5pvtp5l5yt66u50ctrp Data {"version":"0.1.3","timestamp":"1614359551","space":"staking-testnet","type":"vote","payload":{"proposal":"QmZrKwfYV6xXb52xupHRpH52tywANJBhrPBPUtJjGDxgkL","choice":1,"metadata":{}}} Password ••••••••• Reject Approve invalid hexidecimal string (arg="value", value="{\"version\":\"0.1.3\",\"timestamp\":\"1614359551\",\"space\":\"staking-testnet\",\"type\":\"vote\",\"payload\":{\"proposal\":\"QmZrKwfYV6xXb52xupHRpH52tywANJBhrPBPUtJjGDxgkL\",\"choice\":1,\"metadata\":{}}}", version=#version)

2021-02-27 08 07 49

sophoah commented 3 years ago

Do you still have the issue @yelllowsin if not let's close that one

yelllowsin commented 3 years ago

Do you still have the issue @yelllowsin if not let's close that one

Hey @sophoah can you please create a new proposal on testnet so I can test this again? Unfortunately I don't have a validator running, so can't create at the moment. Thanks

yelllowsin commented 3 years ago

Tested this on latest One Wallet version and could not reproduce the issue anymore, seems to be fixed. Tested voting on: https://gov.harmony.one/#/staking-testnet/proposal/QmeSm7x9JoE1J49ERj8x39tb2Jwbi27wh4WsBXoceSkp3p