-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: Add EIP-6963 Provider #7541
Conversation
# Conflicts: # package.json # yarn.lock
# Conflicts: # .depcheckrc.yml # package.json # yarn.lock
# Conflicts: # scripts/build-inpage-bridge.sh
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
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.
LGTM
The assumption here is that Mobile Flask is built off a different branch, thus no build time variables are needed to determine the EIP6963 provider name, icon, and rdns values |
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7541 +/- ##
=======================================
Coverage 34.61% 34.61%
=======================================
Files 1019 1019
Lines 27193 27193
Branches 2218 2218
=======================================
Hits 9413 9413
Misses 17289 17289
Partials 491 491 ☔ View full report in Codecov by Sentry. |
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.
LGTM
Kudos, SonarCloud Quality Gate passed! |
Pulled down Simulator.Screen.Recording.-.iPhone.12.Pro.-.2023-10-31.at.12.42.38.mp4 |
Description
Bump
@metamask/providers
to v13.1.0 which includes EIP6963 support. Pass necessary provider info values to the inpage bridge script to properly initialize the provider.Manual testing steps
Screenshots/Recordings
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/869
Pre-merge author checklist
Pre-merge reviewer checklist