-
-
Notifications
You must be signed in to change notification settings - Fork 235
chore: update accounts/snaps deps #5871
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
Conversation
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
ce31ef4
to
29e04f1
Compare
29e04f1
to
e49ec95
Compare
Adding the |
@@ -65,7 +65,7 @@ const createMockedHandler = () => { | |||
}), | |||
); | |||
const getAccounts = jest.fn().mockReturnValue([]); | |||
const response: PendingJsonRpcResponse<Json> = { | |||
const response: PendingJsonRpcResponse = { |
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.
Most of those changes are required because of @metamask/utils
dedupe. The default type parameter for those is Json
already, resulting in a eslint
warning if we specify it.
See:
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!
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.
CO review for wallet-api-platform-engineers team, only typing changes which make sense. LGTM 👍🏾 🚀
## @metamask/chain-agnostic-permission ## [0.7.1] ### Changed - Bump `@metamask/keyring-internal-api` to `^6.2.0` ([#5871](#5871)) - Bump `@metamask/controller-utils` to `^11.10.0` ([#5935](#5935)) - Bump `@metamask/network-controller` to `^23.6.0` ([#5935](https://github.com/MetaMask/core/pull/5935),[#5882](https://github.com/MetaMask/core/pull/5882)) - Change `caip25CaveatBuilder` to list unsupported scopes in the unsupported scopes error ([#5806](#5806)) ### Fixed - Fix `isInternalAccountInPermittedAccountIds` and `isCaipAccountIdInPermittedAccountIds` to correctly handle comparison against `permittedAccounts` values of the `wallet:<namespace>:<address>` format ([#5980](#5980))
Explanation
Updating
accounts
deps + aligningsnaps
deps as well.The main update here is
@metamask/eth-snap-keyring
which introduce the newKeyringRequest.origin
field. This new breaking change is associated with the Snaps platform version: 7.0.0 (@metamask/[email protected]
).References
N/A
Changelog
N/A
Checklist