Skip to content
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 session key executor #367

Merged
merged 2 commits into from
Jan 17, 2024
Merged

feat: add session key executor #367

merged 2 commits into from
Jan 17, 2024

Conversation

moldy530
Copy link
Collaborator

@moldy530 moldy530 commented Jan 16, 2024

Pull Request Checklist


PR-Codex overview

Focus of the PR:

This PR focuses on adding a SessionKeyExecutor to the accounts package.

Detailed summary:

  • Added SessionKeyExecutor to index.ts in the accounts package.
  • Imported necessary types from @alchemy/aa-core and viem.
  • Exported SessionKeyExecutor as an Executor function.
  • Implemented encodeExecute and encodeBatchExecute functions in SessionKeyExecutor.
  • Used encodeFunctionData to encode function data with the SessionKeyPluginAbi.
  • Handled the case when the account is not connected to an owner.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@moldy530 moldy530 force-pushed the moldy/session-key-executor branch from 06498d0 to d2ee43c Compare January 16, 2024 23:02
@denniswon denniswon force-pushed the denniswon/msca-plugingen-getcontract branch from db3e488 to cd6183f Compare January 17, 2024 00:47
@denniswon denniswon force-pushed the denniswon/msca-plugingen-getcontract branch from cd6183f to ed60cfc Compare January 17, 2024 00:49
@moldy530 moldy530 force-pushed the moldy/session-key-executor branch from d2ee43c to 2afd56a Compare January 17, 2024 16:17
Base automatically changed from denniswon/msca-plugingen-getcontract to main January 17, 2024 16:57
@denniswon denniswon merged commit 0da4c76 into main Jan 17, 2024
2 checks passed
@denniswon denniswon deleted the moldy/session-key-executor branch January 17, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants