-
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
docs: update broken link of consensus client analysis #12412 #12548
docs: update broken link of consensus client analysis #12412 #12548
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.
Looks good! Just added the same change for the non-English pages... thanks @gndelia!
@all-contributors please add @gndelia for bug fix |
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. |
great! Thank you, I did not notice the translated pages. My bad! Thank you for appending the commit 🎉 |
The requirements section in the
Run a node
docs have a section with the following medium link which was broken. This PR fixes that link with a mirror one (content is exactly the same)Description
Replaced
https://medium.com/@migalabs/analysis-of-ethereum-2-consensus-clients-dfede8e0145e
withhttps://mirror.xyz/0x934e6B4D7eee305F8C9C42b46D6EEA09CcFd5EDc/b69LBy8p5UhcGJqUAmT22dpvdkU-Pulg2inrhoS9Mbc
. You can check in web.archive.org that the content is the sameRelated Issue