Skip to content

WIP build optimizations on non-prod envs#15158

Closed
pettinarip wants to merge 8 commits into
devfrom
build-only-default-locale
Closed

WIP build optimizations on non-prod envs#15158
pettinarip wants to merge 8 commits into
devfrom
build-only-default-locale

Conversation

@pettinarip
Copy link
Copy Markdown
Member

Description

Unifies the dynamic pages logic (slug, docs, tutorials) to simplify the build strategy in one place.

New strat:

  • full static build on prod envs
  • only default locale static build on non-prod envs (the rest is generated on demand)

@github-actions github-actions Bot added the tooling 🔧 Changes related to tooling of the project label Mar 24, 2025
@pettinarip pettinarip force-pushed the build-only-default-locale branch from be6798f to 3c454ad Compare March 24, 2025 10:21
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 24, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 65fb846
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/67eed82853ab2100082587ae
😎 Deploy Preview https://deploy-preview-15158--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: 40 (🔴 down 24 from production)
Accessibility: 95 (🟢 up 1 from production)
Best Practices: 91 (🔴 down 7 from production)
SEO: 98 (no change from production)
PWA: 59 (🟢 up 18 from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@pettinarip pettinarip force-pushed the build-only-default-locale branch from 3c454ad to f2b4bc6 Compare March 24, 2025 13:42
Copy link
Copy Markdown
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.

👍 Looks great! Working well locally; consider it approved. @pettinarip Will leave it up to you if you want to merge app-router-pages (#15114) target branch first, or pull this into that PR.


const USE_MOCK_DATA = process.env.USE_MOCK_DATA === "true"

if (USE_MOCK_DATA) {
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.

nice, just noting a similar change was made in #15160 so there may be a tiny conflict here

Base automatically changed from app-router-pages to dev March 28, 2025 19:51
@github-actions github-actions Bot added the config ⚙️ Changes to configuration files label Apr 1, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2025

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions Bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label May 4, 2025
@wackerow wackerow removed the Status: Stale This issue is stale because it has been open 30 days with no activity. label Aug 24, 2025
@wackerow
Copy link
Copy Markdown
Member

#15160 has been merged... if we still want this just need to clear conflicts, update name and flip to ready for review and we'll get it in 🙏

@pettinarip pettinarip closed this Sep 5, 2025
@github-actions github-actions Bot added the abandoned This has been abandoned or will not be implemented label Sep 5, 2025
@pettinarip pettinarip deleted the build-only-default-locale branch September 5, 2025 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

abandoned This has been abandoned or will not be implemented config ⚙️ Changes to configuration files tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants