Skip to content

Conversation

@rahimira2
Copy link

Description

Related Issue

@netlify
Copy link

netlify bot commented Nov 25, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit e1ea23e
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/692565e30e10360008b5b113
😎 Deploy Preview https://deploy-preview-16693--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 48 (🔴 down 11 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (🔴 down 7 from production)
SEO: 91 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

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

@github-actions github-actions bot added the content 🖋️ This involves copy additions or edits label Nov 25, 2025
@corwintines
Copy link
Member

corwintines commented Nov 27, 2025

Any reason for this big rewrite @rahimira2? On a quick skim, this looks like the same content just reformatted more or less.

This would have large implications on the translations so I don't want to bring in without hearing your reason for rewriting first.

@mnelsonBT would also be curious what you think about this.

@mnelsonBT
Copy link
Contributor

Any reason for this big rewrite @rahimira2? On a quick skim, this looks like the same content just reformatted more or less.

This would have large implications on the translations so I don't want to bring in without hearing your reason for rewriting first.

@mnelsonBT would also be curious what you think about this.

Thanks @corwintines - appreciate you calling out the translation considerations for updating this documentation.

Some of these edits do help improve the page's clarity and format (for ex. I think it's great to add the bulleted list of resources under Prerequisites). I agree with you though that keeping these foundational docs translate-able is top priority.
Some changes will be more difficult to translate out of English; removing non-technical descriptors to achieve brevity also removes the context that helps technical jargon translate. A few of these edits also remove background context that is helpful to provide in this "Foundational topics" non-advanced docs section.

Any info you have here on rationale/goals would be helpful @rahimira2 , thanks!

I can review to suggest changes on a few spots where it would be helpful to bring back original context or language. It would be helpful to have your feedback/goals in mind when doing so.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants