superhero-com / superhero-wallet

Superhero is a multi-blockchain wallet to manage crypto assets and navigate the web3 and DeFi space. Currently supporting Bitcoin, Ethereum and æternity blockchains.
https://wallet.superhero.com
ISC License
40 stars 38 forks source link

feat/airgap integration #2983

Closed martinkaintas closed 4 months ago

martinkaintas commented 7 months ago

closes #2866

github-actions[bot] commented 7 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 7 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 7 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 7 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 7 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 6 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

Liubov-crypto commented 5 months ago
  1. I've created multisig vault using airgap vault account as a first signer, after that I proposed a Tx from this superhero vault. I got an confirmation popup that Tx was successful but at the end no pending Tx in my sh wallet. Also I see that on the final step confirmation sign modal is missing if I compare it with multisig vault created from regular ae accounts.

for some reason I can't attach the video((

  1. This is known issue for regular multisig accounts, reproducible also here:

multi nj

  1. When I want to set name by default I will get an modal error, but name will be set after:

def name

  1. Is the text alignment correct here? it looks located close to the edges. @onvisions could you please help me here?

align

  1. I found that QR is blinking in FFweb/ extension and it's hard to scan it from airgap vault:

https://github.com/superhero-com/superhero-wallet/assets/69896204/98d96155-b398-4a81-9002-9123c32c4a19

at the end qr code has been disappeared: create vault

  1. Also found some strange issues in console related to scan qr. Scan qr was not working for me in ff web and extension: ffex

ffe

I see this issue also on prod in console (probably my local issue): sc err

on dev branch FF ext from 05-Jun-2024 08:59 4917797 commit f8edee5:

dev

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

Liubov-crypto commented 5 months ago

Still reproducible for me:

  1. QR scan isn't working in FF.
  2. QR is blinking in FFweb/ extension (actually it's blinking in Chrome too, but I see that in FF delay in blinking is bigger) and it's hard to scan it from airgap vault and at the end qr code has been disappeared.
  3. Sometimes when I want to create vault and choose 2 signer - Create multisig vault button will not respond to me in chrome extension (idk why, probably my pc is overloaded and heated):

https://github.com/superhero-com/superhero-wallet/assets/69896204/612dd0a0-14e7-4971-ab6f-31823b0af13b

  1. do we changed design qr code here? it doesn't fit for the whole box: qr

@onvisions could you please confirm it?

  1. This issue can be reproducible from time to time, especially when I'm proposing a Tx from vault with 0 balance: I've created multisig vault using airgap vault account as a first signer, after that I proposed a Tx from this superhero vault. I got an confirmation popup that Tx was successful but at the end no pending Tx in my sh wallet. Also I see that on the final step confirmation sign modal is missing if I compare it with multisig vault created from regular ae accounts.
github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

Liubov-crypto commented 5 months ago

I've created new wallet for testing.

ad 1. Fixed partly. Scan is working in web wallet but not working in FF extension. sc err

ad 2. QR is blinking in FF: qr code ff qrc

ad 3. - not reproducible for me in new created wallet

ad 4. - updated and looks like it, but not in all places: sc1

but please check all buttons in extension when created vault path and propose / revoke multisig tx:

tr

trs

propose tx mod

ad 5. - not reproducible for me

github-actions[bot] commented 5 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

Liubov-crypto commented 5 months ago

Not a big issue, but when I want to propose multisig tx on some step there will be such qr:

propose multisig

as we agree ad 1 and ad2 will be resolved separately. The rest is LGTM.

github-actions[bot] commented 4 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts

github-actions[bot] commented 4 months ago

Deployed to feat-airgap-integration.wallet.z52da5wt.xyz, artifacts