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 comments for migrator #34

Merged
merged 1 commit into from
Nov 28, 2023
Merged

add comments for migrator #34

merged 1 commit into from
Nov 28, 2023

Conversation

YouStillAlive
Copy link
Member

No description provided.

Copy link

Gas Test Report 📊

The gas test results are as follows:

DelayVaultProvider

Contract Method Min Max Avg # calls
DelayVaultProvider createNewDelayVault(address,uint256[]) 306415 362340 336235 45
DelayVaultProvider createNewDelayVaultWithSignature(address,uint256[],bytes) - - 347507 1
DelayVaultProvider upgradeType(uint8) - - 72915 2

Old DelayVault

Contract Method Min Max Avg # calls
DelayVault CreateVault(address,uint256,uint256,uint256,uint256) 185866 265905 228770 7
DelayVault + DelayVaultMigrator Withdraw(address) 362471 364711 363218 3

DelayVaultMigrator

Contract Method Min Max Avg # calls
DelayVaultMigrator finalize(address) - - 76935 2
DelayVaultMigrator fullMigrate() 364656 371901 368279 2
DelayVaultMigrator withdrawTokensFromV1Vault() 356815 359615 358215 2

Deployments

Contract Min Max Avg % of limit
DelayVaultMigrator 2213390 2213402 2213401 1.7 %
DelayVaultProvider 3791059 3791083 3791075 2.9 %

@Lomet Lomet merged commit f23b510 into master Nov 28, 2023
3 checks passed
@Lomet Lomet deleted the delay-migrator-doc branch November 28, 2023 10: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.

2 participants