Skip to content

fix: rm Promise/await page param type#16316

Merged
corwintines merged 3 commits into
devfrom
page-params
Sep 30, 2025
Merged

fix: rm Promise/await page param type#16316
corwintines merged 3 commits into
devfrom
page-params

Conversation

@wackerow
Copy link
Copy Markdown
Member

Description

  • refactor: use Lang inside PageParams type, implement throughout page param props
  • fix: Remove Promise/await for page & generateMetadata param types (Next v14)
  • Adds type keyword for type imports on these pages
  • Uses { locale: string } type for generateMetadata (See explanation comment)

Related Issue

None filed

refactor: use Lang inside PageParams, implement throughout pages
@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 19, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 8a82f12
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/68db552b358d1a00080a0221
😎 Deploy Preview https://deploy-preview-16316--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: 46 (🔴 down 9 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (🔴 down 8 from production)
SEO: 92 (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 the tooling 🔧 Changes related to tooling of the project label Sep 19, 2025
@corwintines corwintines merged commit 68b91de into dev Sep 30, 2025
6 of 7 checks passed
@corwintines corwintines deleted the page-params branch September 30, 2025 03:58
This was referenced Oct 9, 2025
This was referenced Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants