search

ethereum / ethereum-org-website

Ethereum.org is a primary online resource for the Ethereum community.
https://ethereum.org/
MIT License
4.89k stars 4.67k forks source link

Update node syncing content [Fixes #12003] #13201

Closed m4sterbunny closed 1 day ago

m4sterbunny commented 1 week ago

Description

Execution client summary table did not link users to sync types Sync types were poorly identified and incomplete

Small proofs including:

Issue

Fixes #12003

netlify[bot] commented 1 week ago

Deploy Preview for ethereumorg ready!

Name Link
Latest commit 8398b1a3ccef234e0cf5a16bbacd15f68265a40f
Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/667d25a4797a1e000892b995
Deploy Preview https://deploy-preview-13201--ethereumorg.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse		 7 paths audited
Performance: 42 (🔴 down 7 from production)
Accessibility: 92 (no change from production)
Best Practices: 83 (🔴 down 9 from production)
SEO: 93 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

m4sterbunny commented 1 week ago

I think I need to update Snap further for better detail: there are 2 things to sync Blockchain World state Snap sync: Blockchain from genesis, World state from trusted checkpoint

wackerow commented 2 days ago

Quick housekeeping notes...

"12003(docs):fixes as per issue"

Issue Names: I see you're referencing an issue (12003). By convention, we'll place [Fixes #12003] at the end of the issue name. Please try to start with a brief summary title for your changes. Ie. "Update node syncing content [Fixes #12003]` ("fixes as per issue" isn't very helpful when skimming through issue names 😆 )

More on auto-closing stuff

image

Issue Body: Try to use the default template to get started. Under a ## Description you can describe in more detail your changes, then under ## Relate issue you can add - Fixes #12003 to cleanly link the PR to the issue it's fixing.

Thanks!

m4sterbunny commented 2 days ago

I think I resolved all @wackerow PTAL Apologies for poor hygiene, been lone writer for too long >> will learn to play nicely with others again

wackerow commented 1 day ago

@all-contributors please add @m4sterbunny for content

allcontributors[bot] commented 1 day ago

@wackerow

I've put up a pull request to add @m4sterbunny! :tada: