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

Updated contract addresses across all chains #11

Merged
merged 6 commits into from
Jun 14, 2024

Conversation

kaushalrajbacancy
Copy link
Contributor

Description

  • The addresses of the contracts have been updated on all chains.
  • The installModule and uninstallModule functions have been enhanced to allow direct module installation and uninstallation from within the functions, providing a return of the UserOp Hash.
  • The DataUtils module has been removed from the code.
  • The code supporting ZeroDev and SimpleAccount has been removed.

Types of changes

What types of changes does your code introduce?

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Other (please describe):

Further comments (optional)

Tested the deployment of the account on testnets :

InstallModule :

UninstallModule :

Note: Not tested on any mainnets

Copy link

linear bot commented Jun 13, 2024

Copy link
Member

@ch4r10t33r ch4r10t33r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cryptonoyaiba
Copy link

XDC Apothem deployments (non-deterministic)

ModularEtherspotWallet: 0x15A58094F4DE350ff7F3Ea94B5051a8a4c938Ad5
ModularEtherspotWalletFactory: 0x3aa509fd07B09d5a8b944c96ae6eE767fe75C465 - STAKED 0.1
Bootstrap: 0xCF9bf036D63641003566FD8B10817Dd7168ac5Cb
MultipleOwnerECDSAValidator: 0x61a5eb96F597c72cd24a32D4e241fCA5b22f3dDb
ERC20SessionKeyValidator: 0x70c7169328B2f800BBBdC31269193D6D95C7CdB5

Copy link
Member

@0xSulpiride 0xSulpiride left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kaushalrajbacancy kaushalrajbacancy merged commit d2da7f6 into master Jun 14, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants