-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Phil/set up eth2 translations 2 #1950
Phil/set up eth2 translations 2 #1950
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! Only feedback is my one comment, please refactor to use that new utility method. Thanks!
@@ -93,29 +98,48 @@ const Eth2BeaconChainActions = ({ data }) => { | |||
<StyledCardContainer> | |||
<StyledCardLeft | |||
emoji=":money_with_wings:" | |||
title="Become a staker" | |||
description="Staking is live! If you want to stake your ETH to help secure the network, make sure you’re aware of the risks." | |||
title={intl.formatMessage({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For these uses of intl.formatMessage
let's refactor to use our new utility function translateMessageId
:
https://github.com/ethereum/ethereum-org-website/blob/dev/src/utils/translations.js#L56
Moving forward this will save us from the repetition of adding a defaultMessage
.
More context on the reasoning here: #1940
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Set up Eth2 components for translations. [Fixes #1937]
Description
Set up translations using gatsby-plugin-intl for the following pages:
Related Issue