blocknative / web3-onboard

Client library to onboard users to web3 apps
https://onboard.blocknative.com/
MIT License
829 stars 488 forks source link

Update Capsule Modal #2150

Closed vr2amesh closed 4 months ago

vr2amesh commented 4 months ago

Description

PLEASE NOTE- Checklist must be complete prior to review.

Checklist

Docs Checklist

If this PR includes changes to add an injected wallet or SDK wallet module:

Please complete the following using the internal demo package. To run this demo use the command yarn && yarn dev to get the project running at http://localhost:8080/

Tests with demo app (injected)

Tests with demo app (SDK)

leightkt commented 4 months ago

@vr2amesh I was able to connect a capsule wallet to the demo, but not complete any of the checklist functions listed: send transaction switch chains sign message sign typed message

can you take a look?

vr2amesh commented 4 months ago

@vr2amesh I was able to connect a capsule wallet to the demo, but not complete any of the checklist functions listed: send transaction switch chains sign message sign typed message

can you take a look?

Will take a look now

vr2amesh commented 4 months ago

@leightkt Hi there, I just tested it now, and I was able to do all of those checklist functions. What are you seeing on your end?

leightkt commented 4 months ago

@vr2amesh I connect my wallet and can switch chains- but when I click sign transaction or send transaction- nothing happnes

vr2amesh commented 4 months ago

@leightkt What do you see in the console? The stuff that comes up there is consistent with the output we get from other wallets as well. I don't think the signTransaction is supported in the demo app. I confirmed that the behavior of Capsule wallet is consistent with those other ones/

leightkt commented 4 months ago

@vr2amesh ok so I see a console log of the signed message- I was expecting some sort of UI with Capsule. I was also able to confirm sending a transaction- again, is there no UI for these functions? (just want to make sure I'm understanding correctly).

vr2amesh commented 4 months ago

@leightkt Ah! I understand now. There isn't a UI for these functions. We're meant to be an embedded wallet with minimal UI, so this is expected behavior

leightkt commented 4 months ago

@vr2amesh got ya- thank you for confirming and sorry for the confusion! overall- this PR is looking good. Can you please remove the capsule/yarn.lock file, the update to PORT in package.json and merge in the most recent web3onboard develop branch? I can then get this approved/merged.

vr2amesh commented 4 months ago

@leightkt Absolutely will do! Thank you so much

socket-security[bot] commented 4 months ago

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request