Skip to content

Link to TDS blog post #15816

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

Merged
merged 2 commits into from
Jul 15, 2025
Merged

Link to TDS blog post #15816

merged 2 commits into from
Jul 15, 2025

Conversation

pettinarip
Copy link
Member

No description provided.

@github-actions github-actions bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels Jul 12, 2025
Copy link

netlify bot commented Jul 12, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 98062dd
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6876a141ff088f0008b38c28
😎 Deploy Preview https://deploy-preview-15816--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: 53 (🟢 up 5 from production)
Accessibility: 96 (🟢 up 2 from production)
Best Practices: 91 (🔴 down 1 from production)
SEO: 99 (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.

@@ -7,7 +7,8 @@
"page-trillion-dollar-security-hero-paragraph-2": "But for Ethereum to succeed in the next phase of global adoption, there are still many improvements that must be made. To achieve our community's ambitions, Ethereum must grow into an ecosystem where:",
"page-trillion-dollar-security-hero-paragraph-3": "Billions of individuals are each comfortable holding more than $1000 onchain, collectively amounting to trillions of dollars secured on Ethereum.",
"page-trillion-dollar-security-hero-paragraph-4": "Companies, institutions, and governments are comfortable storing more than 1 trillion dollars of value inside a single contract or application, and are comfortable transacting in comparable amounts.",
"page-trillion-dollar-security-hero-paragraph-5": "The Trillion Dollar Security (1TS) project is an ecosystem-wide effort to upgrade Ethereum's security. This report is the first deliverable of the 1TS project. Over the last month, we have gathered feedback from users, developers, security experts, and institutions about where they see the biggest challenges and areas for improvement. Thank you to the hundreds of people and dozens of organizations who have taken the time to share your insights with us.",
"page-trillion-dollar-security-hero-paragraph-5": "The <a>Trillion Dollar Security (1TS)</a> project is an ecosystem-wide effort to upgrade Ethereum's security. This report is the first deliverable of the 1TS project. Over the last month, we have gathered feedback from users, developers, security experts, and institutions about where they see the biggest challenges and areas for improvement. Thank you to the hundreds of people and dozens of organizations who have taken the time to share your insights with us.",
"page-trillion-dollar-security-hero-paragraph-5-link": "https://blog.ethereum.org/2025/05/14/trillion-dollar-security",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"page-trillion-dollar-security-hero-paragraph-5-link": "https://blog.ethereum.org/2025/05/14/trillion-dollar-security",

Comment on lines 111 to 113
href={t(
"page-trillion-dollar-security-hero-paragraph-5-link"
)}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
href={t(
"page-trillion-dollar-security-hero-paragraph-5-link"
)}
href="https://blog.ethereum.org/2025/05/14/trillion-dollar-security"

Any reason we can't place the link directly in here? No need for the link to be translated

@@ -104,7 +104,19 @@ const TdsPage = async ({ params }: { params: Promise<{ locale: Lang }> }) => {
/>
</div>

<p>{t("page-trillion-dollar-security-hero-paragraph-5")}</p>
<p>
{t.rich("page-trillion-dollar-security-hero-paragraph-5", {
Copy link
Member

Choose a reason for hiding this comment

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

🔥

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for this! Hadn't figured this out yet, but this solves the server-side htmr challenges, and also allows us to inject any variable this way using variations of a <span></span>.

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.

Went ahead and made that change.. cc: @pettinarip gonna pull this in, but let me know if you disagree with that adjustment—can patch if needed

@wackerow wackerow merged commit 357b883 into dev Jul 15, 2025
8 of 9 checks passed
@wackerow wackerow deleted the tds-link branch July 15, 2025 19:02
This was referenced Jul 16, 2025
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.

2 participants