Skip to content

Conversation

corwintines
Copy link
Member

@corwintines corwintines commented Aug 21, 2025

seo - internal images are broken (category in semrush)
seo - internal links are broken (category in semrush)
seo - duplicate meta description (category in semrush)
seo - returned 4XX status code (category in semrush)
seo - incorrect hreflang links (category in semrush)
seo - hreflang conflicts wihtin page source code (category in semrush)
seo - incorrect pages found in sitemap.xml (category in semrush)
seo - duplicate title tags (category in semrush)
seo - duplicate content issues (category in semrush)

Copy link

netlify bot commented Aug 21, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit fb5013e
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/68ac7f6df8d8cc0008cdc2a7
😎 Deploy Preview https://deploy-preview-16100--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: 49 (🟢 up 1 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (no change 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.

@github-actions github-actions bot added config ⚙️ Changes to configuration files content 🖋️ This involves copy additions or edits labels Aug 21, 2025
@github-actions github-actions bot added documentation 📖 Change or add documentation translation 🌍 This is related to our Translation Program labels Aug 21, 2025
@corwintines corwintines mentioned this pull request Aug 22, 2025
1 task
@corwintines corwintines changed the title WIP: SEO Semrush SEO errors Aug 22, 2025
@corwintines corwintines marked this pull request as ready for review August 22, 2025 01:44
Comment on lines +99 to +102
{
protocol: "https",
hostname: "github.com",
},
Copy link
Member

Choose a reason for hiding this comment

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

Hm... curious where this is needed? Small note, but keeping in mind any domain patterns we allow here would allow others to use our next server to optimize their images...

ie, anyone could use our servers to optimize any github-hosted image, using this pattern: https://ethereum.org/_next/image/?url=<any-github-hosted-image-url>&w=<size>&q=<quality>

Copy link
Member

Choose a reason for hiding this comment

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

Not blocking now, but we may want to consider a way to pull the bug-bounty images that this is intended for to use the avatars.githubusercontent.com domain instead.

Copy link
Member

Choose a reason for hiding this comment

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

Comment applies to all non-English files, but we either

  1. must get these updates in Crowdin immediately upon merging to avoid the onslaught of regressions on the 1st of the month (cc: @lukassim), or
  2. remove them to let the Crowdin flow address these corrections

@wackerow wackerow added the Update Crowdin PR introduces changes that need to be updated in Crowdin label Aug 25, 2025
@wackerow
Copy link
Member

cc: @lukassim

Comment on lines +99 to +102
{
protocol: "https",
hostname: "github.com",
},
Copy link
Member

Choose a reason for hiding this comment

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

Not blocking now, but we may want to consider a way to pull the bug-bounty images that this is intended for to use the avatars.githubusercontent.com domain instead.

@wackerow wackerow merged commit 3b9b43c into dev Aug 26, 2025
8 checks passed
@wackerow wackerow deleted the seo branch August 26, 2025 15:26
@corwintines corwintines mentioned this pull request Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙️ Changes to configuration files content 🖋️ This involves copy additions or edits documentation 📖 Change or add documentation translation 🌍 This is related to our Translation Program Update Crowdin PR introduces changes that need to be updated in Crowdin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants