-
Notifications
You must be signed in to change notification settings - Fork 70
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
[NEW][WIP] Add Crypto Wallet - MetaMask Connect Components Kit #92
Conversation
This pull request introduces 5 alerts when merging 92d0657 into 50f7da2 - view on LGTM.com new alerts:
|
Excellent! Plese investigate and add:
Please consider exposing APIs to support:
|
Thank you for the review; I'll look into the support for the Brave wallet and will also expose the APIs currently, only two APIs are used, one to fetch the account and the other to fetch the balance. And will also investigate the other points mentioned above. |
This pull request introduces 3 alerts when merging cbb1942 into 50f7da2 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 0ea549f into 1612542 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 07f8a86 into 1612542 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging bf5053a into 1612542 - view on LGTM.com new alerts:
|
This PR introduces a new component to connect to MetasMask using methods available from the MetaMask API documentation page.
Here is a clip showing the demo of the component.
2022-03-14T07-38-33D0-14_AdobeCreativeCloudExpress.mp4
In the video, the "0" in the dark background is the balance while the hex string is the account identification number.
Thank you!