Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

feat!: add KeyringExecutionContext to user ops methods #253

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

ccharly
Copy link
Contributor

@ccharly ccharly commented Mar 13, 2024

Requires

Description

Adding an additional parameter about the "execution context" (contains only chainId for now) for User operations methods. This will also help computing the CAIP-2 scope/chain-id for underlying calls later on.

Related to

  • metamask/accounts-planning/issues/307

@ccharly ccharly added team-accounts This should be handled by the Accounts Team DO-NOT-MERGE PRs related to this issue should not be merged labels Mar 13, 2024
@ccharly ccharly self-assigned this Mar 13, 2024
@ccharly ccharly requested a review from a team as a code owner March 13, 2024 16:28
@ccharly ccharly marked this pull request as draft March 13, 2024 16:28
@ccharly ccharly changed the title feat!(307): add KeyringExecutionContext to user ops methods #275 feat!(307): add KeyringExecutionContext to user ops methods Mar 13, 2024
@ccharly ccharly force-pushed the feature/307/caip-2-user-ops branch 6 times, most recently from 7f43cda to 31a9ef9 Compare March 15, 2024 10:37
@ccharly ccharly removed the DO-NOT-MERGE PRs related to this issue should not be merged label Mar 15, 2024
@ccharly ccharly marked this pull request as ready for review March 15, 2024 10:37
@ccharly ccharly force-pushed the feature/307/caip-2-user-ops branch from ced56af to b92c51f Compare March 15, 2024 13:08
@ccharly ccharly changed the title feat!(307): add KeyringExecutionContext to user ops methods feat!: add KeyringExecutionContext to user ops methods Mar 15, 2024
@ccharly ccharly force-pushed the feature/307/caip-2-user-ops branch from b92c51f to 5af018d Compare March 15, 2024 13:23
src/caip.ts Outdated Show resolved Hide resolved
src/caip.ts Outdated Show resolved Hide resolved
src/SnapKeyring.test.ts Outdated Show resolved Hide resolved
@ccharly ccharly force-pushed the feature/307/caip-2-user-ops branch 3 times, most recently from 63080ef to e4de14a Compare March 15, 2024 17:28
@ccharly ccharly force-pushed the feature/307/caip-2-user-ops branch from e4de14a to c57c685 Compare March 18, 2024 08:55
@ccharly ccharly added this pull request to the merge queue Mar 18, 2024
Merged via the queue into main with commit 5433d91 Mar 18, 2024
19 checks passed
@ccharly ccharly deleted the feature/307/caip-2-user-ops branch March 18, 2024 10:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team-accounts This should be handled by the Accounts Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants