-
Notifications
You must be signed in to change notification settings - Fork 5k
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
zkSync broken link update #12434
zkSync broken link update #12434
Conversation
WalkthroughThe update involves a straightforward change in the documentation for zk-rollups, specifically adjusting a reference link for ZKSync. The modification shifts the focus from a more detailed developer-oriented page to the main website, potentially to provide a broader overview or direct to a more updated source of information on ZKSync. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
✅ 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.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- public/content/developers/docs/scaling/zk-rollups/index.md (1 hunks)
Check Runs (1)
label completed (2)
Additional comments: 1
public/content/developers/docs/scaling/zk-rollups/index.md (1)
- 243-243: The link for ZKSync has been updated to
https://zksync.io/
. Please ensure this new link is the most appropriate and comprehensive resource for readers seeking information about ZKSync, especially in the context of Ethereum scaling solutions and ZK-rollups. It's important that the link directs users to relevant and up-to-date information.
Update to documentation URL instead of zkSync homepage
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.
Updated broken link to https://docs.zksync.io/
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.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- public/content/developers/docs/scaling/zk-rollups/index.md (1 hunks)
Check Runs (1)
label completed (2)
Additional comments: 1
public/content/developers/docs/scaling/zk-rollups/index.md (1)
- 243-243: The link for ZKSync has been updated from
https://docs.zksync.io/
tohttps://zksync.io/
. This change is appropriate as it directs users to the main homepage of zkSync, which may offer a broader range of information or become the more appropriate destination for users seeking knowledge about zkSync. Ensure that this new link is the most relevant and up-to-date source for zkSync information.
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.
Thanks @isaul-garcia!
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. |
Description
Related Issue
Summary by CodeRabbit