Skip to content

refactor: Developers page to SSR#15629

Merged
pettinarip merged 3 commits into
devfrom
ssr-developers-page
Jun 9, 2025
Merged

refactor: Developers page to SSR#15629
pettinarip merged 3 commits into
devfrom
ssr-developers-page

Conversation

@wackerow
Copy link
Copy Markdown
Member

@wackerow wackerow commented Jun 6, 2025

Description

  • Refactors /developers page to SSR, removing I18nProvider and nested client-side _components/developers.tsx
  • Refactored Callout to Callout + CalloutSSR: SSR version expects final string values for title and description; Callout remains as-named for better backward compatibility, and expects a translation key for title and description to look up the final string client-side, passing that to the SSR version for canonical styling.

Related Issue

Ongoing performance initiatives

wackerow added 2 commits June 6, 2025 14:20
SSR expects final string values for title and description; Callout remains as-named for better backward compatibility, and expects a translation key for title and description to look up the final string client-side, passing that to the SSR version for canonical styling.
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 6, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit b8f1fde
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/684381b7f272e1000843290e
😎 Deploy Preview https://deploy-preview-15629--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: 45 (🔴 down 5 from production)
Accessibility: 95 (🟢 up 1 from production)
Best Practices: 89 (🔴 down 9 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.

fix: common namespace string; fix: duplicate usage of MainArticle (not allowed; contains `id`); fix: inconsistent spacing
Copy link
Copy Markdown
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.

LGTM

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In theory we could just use this one in all the instances, right? we should pass the translated strings.

@pettinarip pettinarip merged commit 9e42bd8 into dev Jun 9, 2025
10 checks passed
@pettinarip pettinarip deleted the ssr-developers-page branch June 9, 2025 16:48
Copy link
Copy Markdown

@tata35tg tata35tg left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants