-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: revert of the revert of "fix(UserMenu): locked metamask not rec… #1732
Conversation
Deploying with Cloudflare Pages
|
@brymut are you able to take a look at this please? we had to back #1667 out of develop as it caused regressions, and develop was not able to connect to metamask. this hdwallet PR was just merged to publish a new version shapeshift/hdwallet#530 you'll need to update to hdwallet |
@0xdef1cafe Pushed bumped hdwallet packages to latest and gave it one more test connecting/waiting for auto-lock, works just as when I was linking hdwallet-metamask |
tested working by ops as per https://discord.com/channels/554694662431178782/974442760247578664/974800402786689045 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have reviewed this before, and just re-tested - it still works as expected.
🍬
yeah gave this another thorough test myself, everything behaves as expected. |
…onnecting (#1667)"
a.k.a the revert of the revert
Requires shapeshift/hdwallet#528 to be published before, and hdwallet deps updated before this goes to review.
Description
Added a button group to separate the dropdown button while wallet is stuck in a loading state when Metamask is locked
Notice
Pull Request Type
Issue (if applicable)
closes #1262
Risk
Testing
Screenshots (if applicable)
Test.locked.mm.mov