Skip to content
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

Roadmap content #9241

Merged
merged 110 commits into from
Mar 3, 2023
Merged

Roadmap content #9241

merged 110 commits into from
Mar 3, 2023

Conversation

jmcook1186
Copy link
Contributor

@jmcook1186 jmcook1186 commented Jan 19, 2023

Description

draft PR containing in-development protocol roadmap content.

Related Issue

#8532
#9242

@github-actions github-actions bot added content 🖋️ This involves copy additions or edits review needed 👀 labels Jan 19, 2023
@konopkja konopkja marked this pull request as ready for review January 26, 2023 13:15
Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jmcook1186 Well done! This was a great read. Left a bunch of small code suggestions and things we need to tidy up, but overall I thought this read very well with good flow to the content.

One thing we'll need to do is set up redirects for the /upgrades/ paths that were removed.

Great job!

@jmcook1186 jmcook1186 marked this pull request as draft January 28, 2023 17:54
@samajammin
Copy link
Member

@jmcook1186 I think you meant to link this PR to this issue? #9242

Copy link
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed UX page


State expiry is still in the research phase and not yet ready to ship. State expiry may well happen later than stateless clients and history expiry because those upgrades make large state sizes easily manageable for the majority of validators.

**Further reading**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@konopkja I think you raised this already, but corect me if I'm wrong. We shouldn't have further reading sections contained within individual headings (with the exception of within collapsable cards).

I'd move these all to the end


Users that run their own nodes do not have to trust third parties to provide them truthful data, and they can interact quickly, privately and permissionlessly with the Ethereum blockchain. However, right now running a node requires some technical knowledge and substantial disk space, meaning many people choose to trust intermediaries instead. There are several upgrades that will make running nodes far easier and far less resource intensive. The way data is stored will be changed to use a more space-efficient structure known as a **Verkle Tree**. Also, with [statelessness](/roadmap/statelessness) or [data expiry](/roadmap/statelessness/#data-expiry), Ethereum nodes will not need to store their own copy of the entire Ethereum state data, drastically reducing hard disk space requirements. Light nodes will offer many of the benefits of running a full node but able to run easily on mobile phones or inside simple browser apps.

<ButtonLink variant="outline-color" to="/roadmap/verkle-trees/">Read about Verkle trees</ButtonLink>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think button links should break sections like this..

cc: @konopkja

Ethereum is scaling through [rollups](/layer-2/#rollups), which batch transactions together and send the output to Ethereum. Even though rollups are up to eight times less expensive than Ethereum Mainnet, it's possible to optimize rollups further to reduce costs for end users. Rollups also rely on some centralized components that developers can remove as the rollups mature.

<InfoBanner mb={8}>
<h4 style="margin-top: 0">Transaction costs</h4>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not blocking, but this creates incorrect heading order by using an H4 here and is an accessibility issue.

cc: @corwintines

template: roadmap
---

Ethereum is scaling through [rollups](/layer-2/#rollups), which batch transactions together and send the output to Ethereum. Even though rollups are up to eight times less expensive than Ethereum Mainnet, it's possible to optimize rollups further to reduce costs for end users. Rollups also rely on some centralized components that developers can remove as the rollups mature.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious why /layer-2/#rollups and not just /layer-2/ here? Aren't all layer 2s under current definition rollups?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to get people straight to the definition of rollup rather than having to skip through the introductory content


## Routes to SSF {#routes-to-ssf}

<ExpandableCard title= "Why can't we have SSF today?" eventCategory="/roadmap/single-slot-finality" eventName="clicked Why can't we hear SSF today?">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why this is a card instead of a section/sub-section?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(not blocking)

@corwintines corwintines marked this pull request as ready for review March 2, 2023 16:50
@corwintines corwintines changed the title draft PR roadmap content Roadmap content Mar 2, 2023
Comment on lines 10 to 13
summaryPoints:
- Understand why account abstraction will make Ethereum easier to use
- Learn about upcoming changes to how users will interact with Ethereum
- Read why account abstraction will help users to secure their assets
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agree with the suggestion from @minimalsm , those should already state the outcome

Comment on lines +93 to +95
## Current progress {#current-progress}

Smart contract wallets are already available, but more upgrades are required to make them as decentralized and permissionless as possible. EIP-4337 is a mature proposal that does not require any changes to Ethereum's protocol, so it is possible that this could be implemented quickly. However, upgrades that alter Ethereum's protocol are currently not in active development, so those changes may take much longer to ship. It is also possible that account abstraction is achieved well enough by EIP-4337 that no protocol changes are ever required.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jmcook1186 "current progress" information is critical information and should be also presented on the main subpages. I would suggest moving this out of the dev docs there on all pages or including in on both.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok i added current progress summaries to the main subpages - lmk what you think

Copy link
Member

@corwintines corwintines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Talked with @jmcook1186 @konopkja @minimalsm, this seems good to go for v1 MVP

@corwintines corwintines merged commit 94f8eb8 into dev Mar 3, 2023
@corwintines corwintines deleted the roadmap-draft branch March 3, 2023 05:29
@corwintines corwintines mentioned this pull request Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants