-
Notifications
You must be signed in to change notification settings - Fork 23
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
Deploy on zk-sync #152
Deploy on zk-sync #152
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Slither found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
I was able to get this deployed, but we were having errors deploying a quest, I noted the error here: zkSync-Community-Hub/zksync-developers#97 It might be because of contract size. We decided not to deploy on ZKSync for now. |
Looks like we are blocked on this because zksync has two issues:
We can't deploy minimal proxies: zkSync-Community-Hub/zksync-developers#91
We can't deploy contracts from other contracts: zkSync-Community-Hub/zksync-developers#97
Also it looks like the foundry version is having issues also: matter-labs/foundry-zksync#113