-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Update zkSync Layer 2 purpose to "universal", description, and Logo #12798
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Hey @zk-Lumi, appreciate this! To my understanding there is now zkSync Lite, and zkSync Era... the first one was the legacy one which was where this entry came from, and still exists (though seems to be transitioning out)
If we're going to update this to use zkSync Era information, I think we should also update the name and logo to match. Let me know if I'm missing something or if you disagree.
Hi @wackerow, For logo: Good point, I updated the logo to the newest branding. |
Ping on this @wackerow 🙏 |
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.
Looks good! Thanks @zk-Lumi!
Congrats, your important contribution to this open-source project has earned you a GitPOAP! Be sure to join the Ethereum.org discord if you are interested in contributing further to the project or have any questions for the team. GitPOAP: 2024 Ethereum.org Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
@all-contributors please add @zk-Lumi for maintenance |
I've put up a pull request to add @zk-Lumi! 🎉 |
Description
Updated zkSync purpose to "universal" + update description to meta description of website