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

fix: update the upgrade scripts to work with AAFactory #223

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

MexicanAce
Copy link
Contributor

Description

Updates to the upgrade.ts script to work with AAFactory.sol updates.

Example usage:

pnpm run upgrade AAFactory --beacon-address 0x7719F498DD5E12577d7C8e741d5873FfC5A12CFc --proxy 0x73CFa70318FD25F2166d47Af9d93Cf72eED48724 --network zkSyncSepoliaTestnet
> zksync-sso-contracts@ upgrade /Users/nick/github/matter-labs/zksync-sso-clave-contracts
> pnpm hardhat upgrade "AAFactory" "--beacon-address" "0x7719F498DD5E12577d7C8e741d5873FfC5A12CFc" "--proxy" "0x73CFa70318FD25F2166d47Af9d93Cf72eED48724" "--network" "zkSyncSepoliaTestnet"

Deploying new implementation of AAFactory contract...
Factory doesn't exist at 0x95005e68E2a1dF01Cc1178f50891D636152122A0!
Requesting contract verification...
src/AAFactory.sol:AAFactory
[WARNING] Since Etherscan is disabled or the API key is missing, verification will default to the ZKSync block explorer.
Your verification ID is: 33275
Successfully verified contract src/AAFactory.sol:AAFactory on the block explorer.
        https://sepolia.explorer.zksync.io/address/0x95005e68E2a1dF01Cc1178f50891D636152122A0#contract
        
New implementation deployed at: 0x95005e68E2a1dF01Cc1178f50891D636152122A0
Upgrading proxy at 0x73CFa70318FD25F2166d47Af9d93Cf72eED48724 to new implementation...
Proxy upgraded successfully

@MexicanAce MexicanAce merged commit 28ce388 into main Dec 13, 2024
3 checks passed
@MexicanAce MexicanAce deleted the nmv/update-scripts-for-factory branch December 13, 2024 16:53
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.

3 participants