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

Add Account Abstraction utils and interfaces #5242

Merged

Conversation

ernestognw
Copy link
Member

@ernestognw ernestognw commented Oct 8, 2024

Smaller version of #5182

This includes:

  • Vendored ERC4337 entrypoint
  • ERC4337Utils
  • ERC7579Utils
  • Extra Packing variants

PR Checklist

  • Tests
  • Documentation
  • Changeset entry (run npx changeset add)

Copy link

changeset-bot bot commented Oct 8, 2024

🦋 Changeset detected

Latest commit: 2073799

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
openzeppelin-solidity Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ernestognw ernestognw added this to the 5.2 milestone Oct 8, 2024
@ernestognw ernestognw marked this pull request as ready for review October 9, 2024 03:57
@ernestognw ernestognw mentioned this pull request Oct 9, 2024
3 tasks
@ernestognw ernestognw requested review from Amxx, arr00 and cairoeth October 9, 2024 05:29
@ernestognw
Copy link
Member Author

The transpilation is failing so the tests-upgradeable is failing. I'll focus on finishing tests for the pending AA-related PRs since this can wait after the audit.

@Amxx Amxx changed the base branch from master to account-abstraction October 9, 2024 13:15
@Amxx Amxx mentioned this pull request Oct 9, 2024
2 tasks
Copy link

socket-security bot commented Oct 10, 2024

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

.codecov.yml Outdated Show resolved Hide resolved
Amxx and others added 6 commits October 10, 2024 21:29
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
@ernestognw
Copy link
Member Author

I'm merging since this is going to the account-abstraction branch and makes it easier to review and iterate the next PR (#5243)

@ernestognw ernestognw merged commit 2aa4828 into OpenZeppelin:account-abstraction Oct 11, 2024
17 checks passed
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