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

post-merge update: block page #7249

Merged
merged 4 commits into from
Aug 30, 2022
Merged

post-merge update: block page #7249

merged 4 commits into from
Aug 30, 2022

Conversation

jmcook1186
Copy link
Contributor

Description

updates block page for post-merge

Related Issue

#7075

@jmcook1186 jmcook1186 changed the title update block page post-merge update: block page Jul 29, 2022
@github-actions github-actions bot added content 🖋️ This involves copy additions or edits review needed 👀 labels Jul 29, 2022
@jmcook1186 jmcook1186 mentioned this pull request Jul 29, 2022
59 tasks
@gatsby-cloud
Copy link

gatsby-cloud bot commented Jul 29, 2022

Gatsby Cloud Build Report

post-merge

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 1m

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.

I think this is a great start! Just left a note about bringing back in the anatomy section for the execution payload. I think this is valuable information to get a better understanding of a block.

Also a few other just spelling/grammar/semantic adjustments that I left code suggestions for.

src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/blocks/index.md Show resolved Hide resolved
@vercel
Copy link

vercel bot commented Aug 26, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
ethereum-org-website ✅ Ready (Inspect) Visit Preview Aug 26, 2022 at 9:27AM (UTC)

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.

This is great! Thanks @jmcook1186! Love having the full list of block contents readily available like this.

@wackerow wackerow merged commit 4cf969d into post-merge Aug 30, 2022
@wackerow wackerow deleted the jc-post-merge-blocks branch August 30, 2022 22:45
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.

2 participants