Merged
Conversation
Replace per-request GitHub API calls with pre-fetched data stored in Netlify Blobs. This eliminates hundreds of API calls during page renders. - Add fetchGitHubContributors scheduled task (weekly) - Use git/trees API for single-request content listing - Fetch commits in parallel batches (20 concurrent) - Store results in data-layer for fast retrieval - Update contributors.ts to read from data-layer - Bump @trigger.dev/sdk to 4.3.3
The previous commit moved GitHub contributor fetching to the data-layer. This removes the now-unused fetchAndCacheGitHubContributors and fetchWithRateLimit functions from gh.ts, along with their imports. Also consolidates CONTENT_DIR and OLD_CONTENT_DIR imports in the new fetcher to use the shared constants.
Untranslated locale pages now correctly signal noindex to crawlers. The canonical tag pointing to the English version is preserved, and follow: true is kept so crawlers still follow links on the page.
Move mobile menu content from server component to client-side lazy-loaded component. This avoids shipping all navigation translations and language display info in the initial RSC payload since the mobile menu is hidden behind a click. Changes: - Add MobileMenuContent.tsx: Client component with the actual menu content - MobileMenuClient.tsx: Uses React.lazy() to load content on first open - MobileMenu/index.tsx: Simplified to just render MobileMenuClient - LvlAccordion.tsx: Remove async keyword that caused repeated Suspense - useLanguagesDisplayInfo.ts: Client hook to generate language info This saves ~82KB from the initial HTML payload by deferring navigation data until the user actually opens the mobile menu.
- Remove 40 truly unused translation keys from common.json - Move page-specific keys to proper namespaces (page-get-eth, page-community-events) - Update code references for relocated keys Addresses review feedback on PR #17662
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Match current English: fix stale bullets, split merged paragraph into 3, add missing MEV-Boost paragraph, fix "closed-source" -> "third-party middleware". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add missing "Track changes" (page-roadmap-release-forkcast) translation to es, zh, ar, hi, id, and it page-roadmap namespace files. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add new bounty hunter 'Cantina' - HIGH 10000 points
Add fixSmartQuotesInJsxAttributes to replace curly quotes (U+201C-201F) with straight ASCII quotes inside JSX component attribute values. 7 new tests, 252 total passing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Fix curly quotes in <YouTube> tag attributes for cs, de, pl, zh glamsterdam translations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
…03-crowdin-translations i18n: automated translation roadmap/glamsterdam (remaining languages)
Arabic: replace Igbo-contaminated file with clean translation Hindi: keep COINBASE opcode in English Indonesian: fix EIP case, slot term, Beacon Chain per glossary Italian: fix false friend (definitivamente -> certamente) Korean: fix broken HTML, org name, Danksharding consistency Vietnamese: fix Ethereum typos, broken links/tags, gas/staker terms Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
…09-crowdin-translations i18n: automated translation page-roadmap.json (remaining languages)
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
bot
added
config ⚙️
feat: add ethskills callout section to /developers
pettinarip
previously approved these changes
Mar 13, 2026
Member
pettinarip
left a comment
pettinarip
left a comment
There was a problem hiding this comment.
Staging Review — v10.27.0
All key changes verified on staging deploy. No issues found.
| Page / Feature | Status | Notes |
|---|---|---|
| Homepage | ✅ Pass | Loads correctly, no A/B test visible (reverted #17742) |
| Footer — lowercase ethereum.org | ✅ Pass | Footer shows "ethereum.org" in lowercase (#17700) |
| Roadmap — Glamsterdam | ✅ Pass | Visible in carousel with "H1 2026", proper artwork (#17730) |
| Grants — ESP Grant Explorer | ✅ Pass | Listed under "Broad Ethereum ecosystem" (#17759) |
| Bug Bounty — Cantina | ✅ Pass | #24 on execution layer with 10,000 points (#17772) |
| Bug Bounty — Revofusion | ✅ Pass | #2 overall (60k pts), appears on both leaderboards (#17767, #17756, #17754) |
| Design & UX — Web3 Design Playbook | ✅ Pass | First item under "Design for web3" (#17727) |
| Run a Node — NiceNode removed | ✅ Pass | Only DappNode and Avado remain (#17726) |
| Search Modal | ✅ Pass | Opens correctly with ErrorBoundary in place (#17755) |
| Breadcrumb casing | ✅ Pass | Correct source value (#17700) |
LGTM — ready to ship 🚀
PR #17662 incorrectly identified "show-more" as unused because TruncatedText.tsx constructed the key dynamically via template literal. This restores the key across all 25 locales and switches to explicit key references so static analysis tools can detect usage.
fix: restore "show-more" translation key removed by #17662
pettinarip
approved these changes
Mar 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
⚡️ Changes
🌐 Translations
📝 Documentation
🔧 Tooling
📦 Dependencies
🦄 Contributors
Thank you @0xMushow, @Dragoon0x, @flatsponge, @jgresham, @minimalsm, @mnelsonBT, @myelinated-wackerow, @pettinarip and @wackerow for the contributions! 🏆