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

feat: Add new words to glossary list #12016

Merged
merged 2 commits into from
Feb 27, 2024
Merged

Conversation

Shiva-Sai-ssb
Copy link
Contributor

Description

I've included six new words and their definitions to the glossary list. The words are layer-1, liquidity, max fee per gas, merkle root, off-chain, and on-chain.

Copy link

netlify bot commented Jan 25, 2024

Deploy Preview for ethereumorg failed.

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/65dcd0a0ae72ab037dec3bbf

@github-actions github-actions bot added content 🖋️ This involves copy additions or edits review needed 👀 translation 🌍 This is related to our Translation Program labels Jan 25, 2024
@minimalsm
Copy link
Contributor

Hey @Shiva-Sai-ssb, thanks for the PR 💪

@konopkja is currently doing some work on the strategic vision of the glossary, so I'll let them review this 😀

@konopkja
Copy link
Contributor

Hello! These look good to me apart from on-chain which we have written a bit different definition but i will add some changes based on your suggestion

@konopkja
Copy link
Contributor

@pettinarip can you merge these except for the onchain one?

Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

The PR lgtm @konopkja.

LMK if these changes to this PR are what you think and we can go ahead and merge it.

public/content/glossary/index.md Show resolved Hide resolved
src/intl/en/glossary.json Outdated Show resolved Hide resolved
@pettinarip pettinarip merged commit 7262601 into ethereum:dev Feb 27, 2024
2 of 6 checks passed
This was referenced Feb 28, 2024
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.

None yet

4 participants