Skip to content

fix: use withBase from runtime#2373

Merged
SoonIter merged 1 commit intoweb-infra-dev:mainfrom
jbroma:fix/use-correct-with-base
Jul 14, 2025
Merged

fix: use withBase from runtime#2373
SoonIter merged 1 commit intoweb-infra-dev:mainfrom
jbroma:fix/use-correct-with-base

Conversation

@jbroma
Copy link
Copy Markdown
Contributor

@jbroma jbroma commented Jul 11, 2025

Summary

This PR fixes an issue where links on the landing page don't include the the basepath (set with base option in config).

routePath passed to HomeHero & HomeFeature components never includes the base so using withBase from @rspress/shared instead of @rspress/runtime will cause the base to be skipped.

Related Issue

n/a

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 11, 2025

Deploy Preview for rspress ready!

Name Link
🔨 Latest commit 1f3fe9a
🔍 Latest deploy log https://app.netlify.com/projects/rspress/deploys/687160051798fa0008906c3d
😎 Deploy Preview https://deploy-preview-2373--rspress.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 81 (🔴 down 15 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (🟢 up 9 from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 11, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit 1f3fe9a
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/68716005697b6b0008bcd3a3
😎 Deploy Preview https://deploy-preview-2373--rspress-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@SoonIter SoonIter enabled auto-merge (squash) July 14, 2025 02:56
@SoonIter SoonIter merged commit 12ca367 into web-infra-dev:main Jul 14, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants