-
Notifications
You must be signed in to change notification settings - Fork 526
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
chore: update the 'metamask-sdk' package to version '0.26.1' #2240
chore: update the 'metamask-sdk' package to version '0.26.1' #2240
Conversation
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
packages/metamask/package.json
Outdated
@@ -60,7 +60,7 @@ | |||
"typescript": "^5.2.2" | |||
}, | |||
"dependencies": { | |||
"@metamask/sdk": "^0.20.5", | |||
"@metamask/sdk": "^0.26.0", |
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.
The package version on line 3 needs an alpha version bump and also needs to be reflected and tested in packages/demo/package.json
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.
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.
After updating the MetaMask SDK package to version 2.1.3-alpha.1
, I encounter an issue when I go to the demo test dapp and try to run yarn
after also updating MetaMask there to "@web3-onboard/metamask": "^2.1.3-alpha.1".
What am I missing in my setup? @Adamj1232
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.
Looks good, thanks!
Description
Update the metamask-sdk package to version '0.26.1'
PLEASE NOTE- Checklist must be complete prior to review.
Checklist
package.json
of the package you have made changes in following semantic versioning and using alpha release taggingyarn check-all
to confirm there are not any associated errorsDocs Checklist
docs/package.json
file (if applicable)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 athttp://localhost:8080/
Tests with demo app (injected)
Tests with demo app (SDK)